Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Be OS Agnostic #793

Open
nwbrettski opened this issue Apr 11, 2024 · 1 comment
Open

Be OS Agnostic #793

nwbrettski opened this issue Apr 11, 2024 · 1 comment

Comments

@nwbrettski
Copy link
Collaborator

The file apps/sdk-node-integration/package.json defines in the scripts.build the linux/unit specific command

rm -rf ./dist && tsup src/index.ts --format cjs,esm --dts

to be executed in the OS shell/terminal. Both the command rm invoked and the way the arguments are passed with -- are OS specific, where Windows uses del for rm and / for -.

Tasks:

  • Find an approach to treat all users regardless of OS as the same
  • Update the readme to remove the Windows user specific steps

Links:

@victhorbi
Copy link
Collaborator

Can be closed after #1406 and being reactive to other architectures/OS afterwards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants