Skip to content

chore(deps): bump sqlalchemy in /templates/api-service-fastapi/recipes - #548

Merged
github-actions[bot] merged 2 commits into
mainfrom
dependabot/pip/templates/api-service-fastapi/recipes/pip-15b89266a7
Nov 5, 2026
Merged

github-actions[bot] merged 2 commits into
mainfrom
dependabot/pip/templates/api-service-fastapi/recipes/pip-15b89266a7

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Nov 1, 2026

Copy link
Copy Markdown
Contributor

Bumps sqlalchemy in /templates/api-service-fastapi/recipes from 2.1.3 to 2.1.6.

Updates sqlalchemy from 2.1.3 to 2.1.6

Release notes

Sourced from sqlalchemy's releases.

2.1.6

Released: October 21, 2026

orm

  • [orm] [bug] Fixed issue where a Session flush that replaced an expired object in the identity map with a newly flushed object of the same primary key would emit a SELECT to refresh the expired object, solely in order to determine whether to emit an "Identity map already had an identity" warning. This could occur when a row loaded within a transaction is rolled back and its primary key value is subsequently reused, and could lead to an AttributeError if the expired object were garbage collected during that load. The check now emits no SQL, and an expired object that is replaced in this way no longer causes the warning to be emitted.

    This change is also backported to: 2.0.56

    References: #13711

  • [orm] [bug] [regression] Fixed regression caused by #13070 where using more than one orm.with_loader_criteria() option for the same entity, where the criteria made use of correlated subqueries referring to that entity, would fail with a RecursionError when compiling.

    References: #13701

  • [orm] [bug] [regression] Fixed regression caused by the new implementation of .params() for statements introduced in #7066, where parameter values passed to the .params() method of an ORM statement created using Select.from_statement() would not be applied when the statement was compiled without the compiled statement cache, such as when the cache was disabled or when the statement was stringified directly, leading to an error that a value is required for the bound parameter.

    References: #13709

orm extensions

  • [bug] [orm extensions] Fixed an issue in the instrumentation_toplevel extension where unrelated classes inheriting from typing.Generic could incorrectly raise a TypeError when using different instrumentation implementations. Pull request courtesy TomasCaruso10.

    This change is also backported to: 2.0.56

    References: #13680

... (truncated)

Commits

Updates sqlalchemy from 2.1.3 to 2.1.6

Release notes

Sourced from sqlalchemy's releases.

2.1.6

Released: October 21, 2026

orm

  • [orm] [bug] Fixed issue where a Session flush that replaced an expired object in the identity map with a newly flushed object of the same primary key would emit a SELECT to refresh the expired object, solely in order to determine whether to emit an "Identity map already had an identity" warning. This could occur when a row loaded within a transaction is rolled back and its primary key value is subsequently reused, and could lead to an AttributeError if the expired object were garbage collected during that load. The check now emits no SQL, and an expired object that is replaced in this way no longer causes the warning to be emitted.

    This change is also backported to: 2.0.56

    References: #13711

  • [orm] [bug] [regression] Fixed regression caused by #13070 where using more than one orm.with_loader_criteria() option for the same entity, where the criteria made use of correlated subqueries referring to that entity, would fail with a RecursionError when compiling.

    References: #13701

  • [orm] [bug] [regression] Fixed regression caused by the new implementation of .params() for statements introduced in #7066, where parameter values passed to the .params() method of an ORM statement created using Select.from_statement() would not be applied when the statement was compiled without the compiled statement cache, such as when the cache was disabled or when the statement was stringified directly, leading to an error that a value is required for the bound parameter.

    References: #13709

orm extensions

  • [bug] [orm extensions] Fixed an issue in the instrumentation_toplevel extension where unrelated classes inheriting from typing.Generic could incorrectly raise a TypeError when using different instrumentation implementations. Pull request courtesy TomasCaruso10.

    This change is also backported to: 2.0.56

    References: #13680

... (truncated)

Commits

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

Bumps [sqlalchemy](https://github.com/sqlalchemy/sqlalchemy) in `/templates/api-service-fastapi/recipes` from 2.1.3 to 2.1.6.


Updates `sqlalchemy` from 2.1.3 to 2.1.6
- [Release notes](https://github.com/sqlalchemy/sqlalchemy/releases)
- [Changelog](https://github.com/sqlalchemy/sqlalchemy/blob/main/CHANGES.rst)
- [Commits](https://github.com/sqlalchemy/sqlalchemy/commits)

Updates `sqlalchemy` from 2.1.3 to 2.1.6
- [Release notes](https://github.com/sqlalchemy/sqlalchemy/releases)
- [Changelog](https://github.com/sqlalchemy/sqlalchemy/blob/main/CHANGES.rst)
- [Commits](https://github.com/sqlalchemy/sqlalchemy/commits)

---
updated-dependencies:
- dependency-name: sqlalchemy
  dependency-version: 2.1.6
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
- dependency-name: sqlalchemy
  dependency-version: 2.1.6
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added the dependencies Dependency updates label Nov 1, 2026
@github-actions
github-actions Bot enabled auto-merge (squash) November 1, 2026 19:11

@yeongseon yeongseon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed the SQLAlchemy 2.1.3 -> 2.1.6 pin change in the FastAPI recipes template. The one-line dependency update remains within sqlalchemy-cubrid's supported sqlalchemy>=2.0,<2.2 range, and the reported CI, docs-sync, PR-title and agent-review workflows pass. No blocking issue found; the normal merge gate should remain authoritative. A Docker Compose API CRUD smoke test would be useful release evidence but is not required to evaluate this one-line dependency bump.

Copy link
Copy Markdown
Contributor

Merge-gate follow-up: I reviewed the SQLAlchemy version bump and attempted a normal squash merge after its reported CI jobs passed. GitHub rejected the merge with Repository rule violations found: 2 of 2 required status checks are expected. The CI Gate (all checks must pass) job and the other shown CI jobs succeeded, so this is not evidence of a failed test; two required check contexts still are not being reported/satisfied for the merge candidate. Please inspect the branch protection/ruleset required-check names and rerun or regenerate the required checks on an updated branch. No protection bypass was attempted.

@yeongseon yeongseon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review summary: this change is limited to the CQRS recipe's SQLAlchemy pin (2.1.3 to 2.1.6). No concrete blocking defect identified in the diff. The dedicated pinned-requirements smoke tests pass against CUBRID 11.2 and 11.4, along with the CI Gate. Broader recipe suites are skipped, so this evidence applies to the targeted recipe. No local test execution was performed.

@github-actions
github-actions Bot merged commit b27dbf7 into main Nov 5, 2026
27 checks passed
@yeongseon

Copy link
Copy Markdown
Contributor

Maintainer review (delegated): one-line SQLAlchemy 2.1.3 to 2.1.6 pin in the recipes template, inside the dialect's supported range. CI Gate and PR title pass on the updated head, so the earlier missing-check rejection no longer applies. No blocking finding. Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependency updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant