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

feat(deps): Bump latest supported sqlalchemy from 1.* to 2.* #1484

Merged
merged 21 commits into from
Jul 20, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 6, 2023

  • Increases the range of supported SQLAlchemy versions
  • Run all existing tests on the latest SQLAlchemy 2.* and add a single new test against SQLAlchemy 1.*. Version is printed in the Pytest output:
============================= test session starts ==============================
platform linux -- Python 3.11.4, pytest-7.4.0, pluggy-1.2.0 -- /home/runner/work/sdk/sdk/.nox/tests-3-11/bin/python
cachedir: .pytest_cache
sqlalchemy: 1.4.49 <--------------------- This is added
rootdir: /home/runner/work/sdk/sdk
configfile: pyproject.toml
testpaths: tests
plugins: snapshot-0.9.0, requests-mock-1.11.0, durations-1.2.0
collecting ... collected 449 items / 2 deselected / 447 selected

📚 Documentation preview 📚: https://meltano-sdk--1484.org.readthedocs.build/en/1484/

> **Note**

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@dependabot dependabot bot force-pushed the dependabot/pip/sqlalchemy-2.0.5.post1 branch from bcdea80 to abdcc57 Compare March 6, 2023 16:20
@dependabot dependabot bot force-pushed the dependabot/pip/sqlalchemy-2.0.5.post1 branch 2 times, most recently from 88fdc5f to 10d7f5f Compare March 6, 2023 19:24
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Mar 14, 2023

A newer version of sqlalchemy exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@edgarrmondragon edgarrmondragon changed the title chore(deps): Bump sqlalchemy from 1.4.46 to 2.0.5.post1 chore(deps): Bump sqlalchemy from 1.4.46 to 2.0.15 Jun 2, 2023
@edgarrmondragon edgarrmondragon force-pushed the dependabot/pip/sqlalchemy-2.0.5.post1 branch from eaa4188 to 6e03345 Compare June 2, 2023 20:47
@edgarrmondragon edgarrmondragon changed the title chore(deps): Bump sqlalchemy from 1.4.46 to 2.0.15 chore(deps): Bump latest supported sqlalchemy from 1.4.46 to 2.0.15 Jun 2, 2023
@edgarrmondragon edgarrmondragon changed the title chore(deps): Bump latest supported sqlalchemy from 1.4.46 to 2.0.15 chore(deps): Bump latest supported sqlalchemy from 1 to 2 Jun 2, 2023
@edgarrmondragon edgarrmondragon changed the title chore(deps): Bump latest supported sqlalchemy from 1 to 2 chore(deps): Bump latest supported sqlalchemy from 1.* to 2.* Jun 2, 2023
@edgarrmondragon edgarrmondragon changed the title chore(deps): Bump latest supported sqlalchemy from 1.* to 2.* feat(deps): Bump latest supported sqlalchemy from 1.* to 2.* Jun 21, 2023
@codecov
Copy link

codecov bot commented Jun 21, 2023

Codecov Report

Merging #1484 (c88a47d) into main (384cff7) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1484   +/-   ##
=======================================
  Coverage   86.68%   86.68%           
=======================================
  Files          59       59           
  Lines        5000     5000           
  Branches      811      811           
=======================================
  Hits         4334     4334           
  Misses        470      470           
  Partials      196      196           
Impacted Files Coverage Δ
singer_sdk/sinks/sql.py 90.17% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@edgarrmondragon edgarrmondragon force-pushed the dependabot/pip/sqlalchemy-2.0.5.post1 branch from e0a8e64 to f2513fe Compare July 11, 2023 03:14
@edgarrmondragon edgarrmondragon force-pushed the dependabot/pip/sqlalchemy-2.0.5.post1 branch from bc6b0f4 to 59ed35c Compare July 11, 2023 20:44
@edgarrmondragon edgarrmondragon force-pushed the dependabot/pip/sqlalchemy-2.0.5.post1 branch from 59ed35c to 221a15c Compare July 13, 2023 18:30
if isinstance(records, list)
else (self.conform_record(record) for record in records)
)
conformed_records = [self.conform_record(record) for record in records]
Copy link
Collaborator

Choose a reason for hiding this comment

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

execute doesn't really support generators, only sequences (i.e. lists, tuples)

@edgarrmondragon edgarrmondragon requested a review from a team July 20, 2023 20:12
@edgarrmondragon edgarrmondragon merged commit b5cc805 into main Jul 20, 2023
28 checks passed
@edgarrmondragon edgarrmondragon deleted the dependabot/pip/sqlalchemy-2.0.5.post1 branch July 20, 2023 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant