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

Adding support for creating templates from .env files. #377

Merged
merged 2 commits into from
Jun 22, 2019

Conversation

mikeshaw-stripe
Copy link
Contributor

Templates can be checked in and will show the environment variables used but wont show the values

I love using this package while writing demos at Stripe. However every now and then I miss a variable in my README.md or a handcrafted .env.template.

This adds a flag to the cli so that you can point it at any .env file and safely create a template from it that doesn't include any of your keys. :)

Templates can be checked in and will show the environment variables used but wont show the values
@cadwallion cadwallion self-assigned this Mar 22, 2019
@cadwallion
Copy link
Collaborator

Thank you for the submission, and apologies for being slow to review this! I'll get this reviewed today. :)

@cadwallion
Copy link
Collaborator

Conceptually I like this. :) Let me put it through its paces this week to see if anything shakes out. I'm going to be cutting a patch-level version release today ahead of this since a feature like this will need a minor version bump.

@mikeshaw-stripe
Copy link
Contributor Author

OK sounds good! Let me know if there is anything I can help with

@thomasfedb
Copy link
Contributor

Would be nice to see this merged @cadwallion

@cadwallion
Copy link
Collaborator

👋 @thomasfedb Sorry to only just now swing back to this. I'm happy to include this if you can fix the merge conflict in the CLI.

@cadwallion cadwallion merged commit dc91d94 into bkeepers:master Jun 22, 2019
cadwallion pushed a commit that referenced this pull request Jun 22, 2019
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.

3 participants