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

improved default python vars #1100

Merged
merged 5 commits into from May 17, 2024

Conversation

iloveitaly
Copy link
Contributor

Reopened this PR with requested documentation.

These look good, but just wondering where these came from

These are generally recommended variables for production. I've curated them over the last year from various python docker files I've worked with.
They are all documented here:

https://docs.python.org/3/using/cmdline.html

This set of environment variables seems to be the best set of default production settings
for most python projects.
@coffee-cup coffee-cup added the release/patch Author patch release label May 9, 2024
Copy link
Contributor

@coffee-cup coffee-cup left a comment

Choose a reason for hiding this comment

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

Thanks! I'll fix the lints in a different PR and then merge this

@iloveitaly
Copy link
Contributor Author

@coffee-cup awesome! Sounds great.

@coffee-cup
Copy link
Contributor

I've pushed 2 commits to fix the tests and linting

@coffee-cup coffee-cup merged commit 3baec23 into railwayapp:main May 17, 2024
95 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/patch Author patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants