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

new server test with precommit hook to secure .env files, set up CI e… #24

Closed

Conversation

jlieb10
Copy link
Contributor

@jlieb10 jlieb10 commented Dec 11, 2018

…nvironment (#3)

@jlieb10
Copy link
Contributor Author

jlieb10 commented Dec 11, 2018

@breezykermo someone with FA contributor status will need to commit with a .env file for Travis to pass

@breezykermo
Copy link
Member

What exactly goes in the .env file in this branch? If possible, I think it would be good to keep the server to just one configuration file, i.e. config.js.

config.js is where the private key is kept currently, and as such I would have thought that it should be the encrypted file sent to the Travis server.

@jlieb10
Copy link
Contributor Author

jlieb10 commented Dec 11, 2018

I'm adding some breaking changes to this PR - will ping when ready

@jlieb10 jlieb10 closed this Dec 14, 2018
@jlieb10 jlieb10 deleted the test-with-precommit-env branch December 14, 2018 15:54
@jlieb10 jlieb10 restored the test-with-precommit-env branch December 14, 2018 15:56
@jlieb10 jlieb10 deleted the test-with-precommit-env branch December 14, 2018 15:57
@jlieb10 jlieb10 restored the test-with-precommit-env branch December 14, 2018 15:58
@jlieb10 jlieb10 deleted the test-with-precommit-env branch December 14, 2018 15:59
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.

2 participants