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 path to diff-check in action.yml #5

Merged
merged 1 commit into from
Jun 14, 2024
Merged

Fix path to diff-check in action.yml #5

merged 1 commit into from
Jun 14, 2024

Conversation

nickcharlton
Copy link
Owner

It was previously assumed that $GITHUB_PATH affected the $PATH lookup but this wasn't actually the case and so the command could never be run.

Instead, this uses $GITHUB_ACTION_PATH (which is the path to the action cached on the runner) to built out a full path for diff-check instead, which is also simpler.

Fixes #4.

It was previously assumed that `$GITHUB_PATH` affected the `$PATH`
lookup but this wasn't actually the case and so the command could never
be run.

Instead, this uses `$GITHUB_ACTION_PATH` (which is the path to the
action cached on the runner) to built out a full path for `diff-check`
instead, which is also simpler.

Fixes #4.
@nickcharlton nickcharlton merged commit 48caa85 into main Jun 14, 2024
1 check passed
@nickcharlton nickcharlton deleted the fix-path branch June 14, 2024 16:04
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.

bin/diff-check: No such file or directory
1 participant