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.

Sign up for GitHub

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

Jump to bottom

Investigate methods to declare expected augur/fauna/… versions #3

Open
tsibley opened this issue Jun 30, 2018 · 0 comments
Open

Investigate methods to declare expected augur/fauna/… versions #3

tsibley opened this issue Jun 30, 2018 · 0 comments

Comments

@tsibley
Copy link
Member

tsibley commented Jun 30, 2018

Not sure what this should look like, but it might be a very useful thing to have. On the other hand, if we're good at preserving backwards compat once augur stabilizes a bit, this may be mostly a non-issue. A topic to think about and discuss.
@tsibley tsibley mentioned this issue Jul 2, 2018
Closed
tsibley added a commit that referenced this issue Jul 2, 2018
@tsibley
wip! Remove credentials from config in favor of environment variables
2d14707 
This is more standard (e.g. follows the 12 factor app pattern) and, as
Trevor noted¹, is encouraged by lots of services like Travis CI and
Heroku and EC2.  It also means credentials are not visible in the output
of `ps`, which is important for running on shared hosts.

Unresolved is how to programmatically describe the requirements on the
"envdir" module.  This is an unresolved question for build requirements
in general.²  It may be that we sidestep that by accessing envdir
through a nextstrain.build module that's always required to be
available, as described in a proposal for refactoring shared functions.³

¹ #4
² #3
³ #2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tsibley