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

Added script to create augur.config.json #356

Merged
merged 1 commit into from
Aug 17, 2019
Merged

Conversation

parthsharma2
Copy link
Member

Created a script that allows you to interactively create augur.config.json. To run the script execute the following command

make config

Signed-off-by: Parth Sharma [email protected]

Signed-off-by: Parth Sharma <[email protected]>
@parthsharma2 parthsharma2 added the NDE Related to the new developer experience label Aug 17, 2019
Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ccarterlandis and @gabe-heim : I am not sure why this is failing the travis build. @parthsharma2 : Can you make sure that you have pulled FROM augur:dev --> parthsharma2:dev in the past day. I suspect this could be the reason its failing. It could also be the changes to the makefile are a problem.

ccarterlandis added a commit that referenced this pull request Aug 17, 2019
@ccarterlandis
Copy link
Contributor

So it took me entirely too long to remember this, but it's failing because we've configured the Travis build to remove our encrypted environment variables from any build that doesn't come from within the main augur repository, as we don't want untrusted builds to have access to our credentials. That's not to say we don't trust you @parthsharma2 - in fact, we very much trust you! It's just a matter of precaution. @sgoggins You also might recall that we ran into this issue back in January for the same reason...my vote is we just adopt the same solution we did last time 😄

ccarterlandis added a commit that referenced this pull request Aug 17, 2019
@sgoggins sgoggins merged commit 966e10a into chaoss:dev Aug 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NDE Related to the new developer experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants