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

Changed to double quotes in npm script #86

Merged
merged 1 commit into from
Sep 8, 2021
Merged

Changed to double quotes in npm script #86

merged 1 commit into from
Sep 8, 2021

Conversation

kaffarell
Copy link
Contributor

Single quotes don't work on windows, so I changed to double quotes and
escaped them with a backslash (should work on all platforms, tried on
windows and linux).

@imRohan imRohan self-requested a review September 6, 2021 13:28
@imRohan imRohan assigned imRohan and kaffarell and unassigned imRohan Sep 6, 2021
@imRohan imRohan added the bug Something isn't working label Sep 6, 2021
@imRohan
Copy link
Owner

imRohan commented Sep 7, 2021

I merged in the latest changes from master which lets the CI action run on pull requests. Could you please squash all of the commits into one please? Thanks!

@kaffarell
Copy link
Contributor Author

👍 Fixed it

Copy link
Owner

@imRohan imRohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The intended changes to package.json look good! 👍 I see there are other changes that you'll need to address, maybe rebase on master since these changes currently exist.

Also, could you please update your commit message to match the PR description? I've added a pull request template here

@imRohan imRohan added Ready for Review Once you've squashed your commits, the PR is now ready for review! and removed bug Something isn't working labels Sep 7, 2021
The single quotes in the package.json file did not work on windows.
I changed to double quotes and escaped them with a backslash. (Tested on
windows and linux).
Resolves: #86
@kaffarell
Copy link
Contributor Author

kaffarell commented Sep 8, 2021

Sorry for the mess, I accidentaly updated the kaffarell/master branch with unrelated changes from the imRohan/master branch (Added the wrong remote url). Reverted these changes and made a new commit with only the intended changes (also following the guidelines). Hope this works!

@imRohan imRohan merged commit 13c0ea3 into imRohan:master Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for Review Once you've squashed your commits, the PR is now ready for review!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants