Skip to content

Modify alembic function for virtual environment support#185

Merged
sichapman merged 2 commits intoLandRegistry:developfrom
jwlademann:alembic_newer_base_images
Mar 6, 2026
Merged

Modify alembic function for virtual environment support#185
sichapman merged 2 commits intoLandRegistry:developfrom
jwlademann:alembic_newer_base_images

Conversation

@jwlademann
Copy link
Copy Markdown
Contributor

Source venv if env var is defined to allow support for newer images (e.g. python 3.13)
Something of a duplicate of #182 since Sarah is off sick currently and we're hitting this issue with our new APIs.

  • What kind of change does this PR introduce (Bug fix, feature, docs update, ...)?
    Source venv if env var is defined to allow support for newer images (e.g. python 3.13)

  • What is the current behavior?
    Alembic alias does not work with newer base images, e.g. python 3.13

  • What is the new behavior (if this is a feature change)?
    Fixes existing feature

  • Does this PR introduce a breaking change? If so, what actions will users need to take in order to regain compatibility?
    No

Checklists

All submissions

  • Have you followed the guidelines in our Contributing document?
  • Is this PR targeting the develop branch, and the branch rebased with commits squashed if needed?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New feature and bug fix submissions

  • Has your submission been tested locally?
  • [n/a] Has documentation such as the README been updated if necessary?
  • [n/a is in bash script] Have you linted your new code (using rubocop and markdownlint), and are not introducing any new offenses?

sichapman and others added 2 commits July 15, 2025 14:18
Source venv if env var is defined to allow support for newer images (e.g. python 3.13)
@jwlademann jwlademann changed the base branch from master to develop March 5, 2026 12:01
@sichapman sichapman merged commit cfa77d3 into LandRegistry:develop Mar 6, 2026
1 check passed
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