Skip to content

Action-specific variable names#51

Merged
CasperWA merged 4 commits intomasterfrom
fix-50-action-specific-var-names
Oct 4, 2021
Merged

Action-specific variable names#51
CasperWA merged 4 commits intomasterfrom
fix-50-action-specific-var-names

Conversation

@CasperWA
Copy link
Copy Markdown
Owner

@CasperWA CasperWA commented Oct 4, 2021

FIxes #50

Prefixes all local bash variable names with PUSH_PROTECTED_ to avoid clashing with workflow-created variables.

Minor updates:

  • Explicitly use string values as default in action.yml.
  • Update pre-commit hook black (no updates after running pre-commit run --all-files).
  • Silence linting for using HTML in README.md.

@CasperWA CasperWA merged commit 37f26e1 into master Oct 4, 2021
@CasperWA CasperWA deleted the fix-50-action-specific-var-names branch October 4, 2021 12:02
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.

Use more action-specific local bash variable names

1 participant