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

fix: Windows runner 'node_modules' is not recognized as an internal or external command #79

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EdgarsGarsneks
Copy link

@EdgarsGarsneks EdgarsGarsneks commented Jan 28, 2024

Summary

Fix proposal for Windows runner 'node_modules' is not recognized as an internal or external command issue resolving #78.

Here is workflow run that executed all os on fork. I was lazy to fix github secret step because of that all jobs fail, but they do execute build and lint successfully, which I hope should be enough evidence 😄

Details

  • Instead of executing ng build and lint by path node_modules/.bin/ng use npx npx ng instead that does not rely on path.
  • Update workflow to test action also on windows and macos

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

Successfully merging this pull request may close these issues.

None yet

1 participant