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

Protect against accidental SOLC_VERSION clash from workflow #22

Merged
merged 1 commit into from
Jul 29, 2022

Conversation

elopez
Copy link
Member

@elopez elopez commented Jul 28, 2022

Users may set SOLC_VERSION in the workflow and cause hard-to-diagnose
problems in the action (see #19). To avoid that
from happening, unset SOLC_VERSION early on. If the user desires
to change the solc version, they can always use the solc-version
action option.

Closes: #19

Users may set SOLC_VERSION in the workflow and cause hard-to-diagnose
problems in the action (see #19). To avoid that
from happening, unset SOLC_VERSION early on. If the user desires
to change the solc version, they can always use the `solc-version`
action option.
@elopez elopez force-pushed the dev-unset-solc-version branch from bf5e8b5 to 7a59832 Compare July 29, 2022 13:28
@elopez elopez merged commit 5c9b113 into dev Jul 29, 2022
@elopez elopez deleted the dev-unset-solc-version branch July 29, 2022 13:28
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