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

removed postgres_password from alembic.ini #9

Merged
merged 8 commits into from
Apr 20, 2019

Conversation

ebreton
Copy link
Contributor

@ebreton ebreton commented Apr 12, 2019

read it from env var instead

This allow to keep all environment variables in the *env files.

@tiangolo
Copy link
Member

This looks great, I'll review it soon.

ebreton and others added 7 commits April 16, 2019 21:36
* Update CRUD utils to use types better.
* Simplify Pydantic model names, from `UserInCreate` to `UserCreate`, etc.
* Upgrade packages.
* Add new generic "Items" models, crud utils, endpoints, and tests. To facilitate re-using them to create new functionality. As they are simple and generic (not like Users), it's easier to copy-paste and adapt them to each use case.
* Update endpoints/*path operations* to simplify code and use new utilities, prefix and tags in `include_router`.
* Update testing utils.
* Update linting rules, relax vulture to reduce false positives.
* Update migrations to include new Items.
* Update project README.md with tips about how to start with backend.
@tiangolo tiangolo merged commit bece399 into fastapi:master Apr 20, 2019
@tiangolo
Copy link
Member

Thanks! Great job!

I just updated the URL string generation to use f-strings.

Sorry I ended up merging the latest changes into your branch 😁

gusevyaroslove pushed a commit to gusevyaroslove/fastapi-template that referenced this pull request Aug 4, 2024
♻️ removed postgres_password from alembic.ini (fastapi#9)
alejsdev pushed a commit that referenced this pull request Dec 19, 2024
Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
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