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

feat: split out the action into action.sh #10

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Conversation

matijs
Copy link
Owner

@matijs matijs commented Feb 8, 2024

Factor out the action into action.sh so it can be shellchecked. Use github.action_path this time so it works properly when used as an action.

@matijs matijs force-pushed the feat/use-script branch 4 times, most recently from eb6220b to a7f74ba Compare February 8, 2024 07:31
Move the action out of action.yml into action.sh so it can be shell
checked. Use `github.action_path` so the action can actually be used as
an action.

Add a shellcheck step to ci.yml
@matijs matijs enabled auto-merge (squash) February 8, 2024 07:33
@matijs matijs merged commit a55b270 into main Feb 8, 2024
2 checks passed
@matijs matijs deleted the feat/use-script branch February 8, 2024 07:33
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.

1 participant