-
Notifications
You must be signed in to change notification settings - Fork 37
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
Parametrize dragon install #703
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks really good to me! Just a couple docstring fix requests and a question about to what extent we should document this, but once the tests pass I think this should be good to go.
@AlyssaCote do you think we need to add the |
@al-rigazzi I believe that's only necessary if we want to ensure the use of HSTA, otherwise the fallback is TCP. So as of now I don't think we need it for tests? But I could be wrong! |
@AlyssaCote yeah, not for tests, you're right. I guess, with the current implementation, if a user wanted to use HSTAs, the |
@al-rigazzi Ohhh good point. I was only thinking of the green checks for our github actions! Yeah you're right we do need a way for users to enable HSTA. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Can't wait to see all the green check marks!!
Parameterize the
smart build --dragon
command to enable specification of a fork/repository and package version--dragon-repo
--dragon-version
Sample usage:
Get latest version from private fork
GH_TOKEN=xxxxx smart build --dragon-repo ankona/dragonfork
Get specific version from private fork
GH_TOKEN=xxxxx smart build --dragon-repo ankona/dragonfork --dragon-version 0.10
Get specific version from public fork
GH_TOKEN=xxxxx smart build --dragon-repo dragonhpc/dragon --dragon-version 0.10
GH_TOKEN=xxxxx smart build --dragon-version 0.10
smart build --dragon-repo dragonhpc/dragon --dragon-version 0.10
smart build --dragon-version 0.10
manual test results
smart build --dragon
smart build --dragon-repo dragonhpc/dragon-nightly
smart build --dragon-repo dragonhpc/dragon-nightly --dragon-version 0.9
GH_TOKEN=xxx smart build --dragon-repo dragonhpc/dragon-nightly --dragon-version 0.9
smart build --dragon-repo dragonhpc/dragon-nightly --dragon-version 0.10
GH_TOKEN=xxx smart build --dragon-repo dragonhpc/dragon-nightly --dragon-version 0.10
smart build --dragon-version 0.10
smart build --dragon-version 0.9
export GH_TOKEN=xxx smart build --dragon-version 0.9
smart build --dragon-repo dragonhpc/dragon
GH_TOKEN=xxx smart build --dragon-repo dragonhpc/dragon
smart build --dragon-repo dragonhpc/dragon --dragon-version 0.9
GH_TOKEN=xxx smart build --dragon-repo dragonhpc/dragon --dragon-version 0.9
smart build --dragon --dragon-version 0.9
smart build --dragon --dragon-version 0.10
smart build -h