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

Add option to skip .env check #487

Closed
wants to merge 1 commit into from
Closed

Conversation

snopoke
Copy link

@snopoke snopoke commented Sep 21, 2023

This would allow deploying without having the .env file generated locally. This is especially useful for automated deploys which may not have access to the secrets e.g. 1Password.

Ideally this option should only be for deploy but the env check is part of the base command.

Side note: I'm new to the repo and Ruby so please let me know if there are test that should be added or doc updates etc.

This would allow deploying without having the `.env` file generated locally. This is especially useful for automated deploys which may not have access to the secrets e.g. in 1Password.
@djmb
Copy link
Collaborator

djmb commented Sep 25, 2023

Thanks for pointing this out @snopoke! I've raised #499 to fix this. We can remove the check entirely as it's generally not required anymore now that the environment is pushed to remote servers in a separate step.

@snopoke snopoke closed this Sep 26, 2023
@snopoke snopoke reopened this Sep 26, 2023
@snopoke snopoke closed this Sep 26, 2023
@snopoke snopoke deleted the patch-1 branch September 26, 2023 07:04
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