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.

Sign up for GitHub

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

Jump to bottom

feat: Prepara código para API do sqlalchemy 2.0 #298

Merged
merged 4 commits into from
Feb 16, 2023
Merged

feat: Prepara código para API do sqlalchemy 2.0 #298

merged 4 commits into from
Feb 16, 2023

Conversation

thulio
Copy link
Collaborator

@thulio thulio commented Feb 15, 2023 •
edited

Esse PR remove os warnings do sqlalchemy

@thulio thulio changed the title feat/prepare to sqlalchemy20 feat: Prepara código para API do sqlalchemy 2.0 Feb 15, 2023
@thulio thulio self-assigned this Feb 15, 2023
@thulio thulio added bug Something isn't working enhancement New feature or request labels Feb 15, 2023
@thulio thulio force-pushed the feat/prepare-to-sqlalchemy20 branch from e26d2c2 to 0e41fd8 Compare February 15, 2023 21:05
@thulio thulio marked this pull request as ready for review February 15, 2023 21:10
@thulio thulio requested review from diogommartins, eduwxyz and lemosflavio and removed request for diogommartins February 15, 2023 21:11
@diogommartins
Copy link
Contributor

Relacionado a https://docs.sqlalchemy.org/en/20/changelog/migration_20.html#select-no-longer-accepts-varied-constructor-arguments-columns-are-passed-positionally

@thulio thulio merged commit dc513f6 into main Feb 16, 2023
@thulio thulio deleted the feat/prepare-to-sqlalchemy20 branch February 16, 2023 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lemosflavio lemosflavio lemosflavio approved these changes

@diogommartins diogommartins diogommartins approved these changes

@eduwxyz eduwxyz Awaiting requested review from eduwxyz

Assignees

@thulio thulio

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@thulio @diogommartins @lemosflavio