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

Adds scenario for if a user does not want to install asdf on Ubuntu #4

Merged
merged 1 commit into from
Feb 9, 2022

Conversation

steffkelsey
Copy link
Owner

Addresses #3

Hole in the test coverage! We are not testing if a user wishes to skip
the asdf install on this playbook (which would just do nothing to the
target instance). The ability to opt-out is a huge selling point for
eng. This adds an automated scenario for Linux. It does not cover macOS
because the macOS tests take almost an order of magnitude longer to run
in GitHub Actions, so leaving macOS off and considering it "good
enough", but the code path is different so there is some risk going
forward.

Addresses #3

Hole in the test coverage! We are not testing if a user wishes to skip
the asdf install on this playbook (which would just do nothing to the
target instance). The ability to opt-out is a huge selling point for
eng. This adds an automated scenario for Linux. It does not cover macOS
because the macOS tests take almost an order of magnitude longer to run
in GitHub Actions, so leaving macOS off and considering it "good
enough", but the code path is different so there is some risk going
forward.
@steffkelsey steffkelsey merged commit 0c57938 into main Feb 9, 2022
@steffkelsey steffkelsey deleted the steff/no-install-asdf-scenario branch February 9, 2022 19:00
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