Skip to content

fix: 9 fix location and pathing to the capedb script and alembic files#10

Merged
thecaffiend merged 21 commits into
mainfrom
9-fix-location-and-pathing-to-the-capedb-script-and-alembic-files
Mar 5, 2026
Merged

fix: 9 fix location and pathing to the capedb script and alembic files#10
thecaffiend merged 21 commits into
mainfrom
9-fix-location-and-pathing-to-the-capedb-script-and-alembic-files

Conversation

@thecaffiend
Copy link
Copy Markdown
Member

@thecaffiend thecaffiend commented Mar 5, 2026

TO TEST

  • follow the setup to create an empty database
  • poetry build this repo to get a wheel built
  • make a test venv with python 3.10 for testing
  • install the wheel into the test venv (e.g. with pip)
  • capedb -c <path to this repo>/cape_cod_db/alembic.ini -x db_url=postgresql:///cape_env_db upgrade head
  • capedb -c <path to this repo>/cape_cod_db/alembic.ini -x db_url=postgresql:///cape_env_db current
  • ensure output is 6001985fea71 (head)

thecaffiend added 15 commits March 4, 2026 15:58
@thecaffiend
moving files around
06fd592
@thecaffiend
changed to support custom cli option for db_url. removed bypass of al…
dfc0eba 
…embic ini
@thecaffiend
fixed some order of operations and removed basic config from logging
034bdc5
@thecaffiend
changed to use config from alembic setup
8a9bfba
@thecaffiend
packages
05a841a
@thecaffiend
removed old includes and added new cli script pathing
623f7b4
@thecaffiend
changed 'alembic' package name (and obvi dir) to 'migrations' so as t…
43912ef 
…o not conflict with actual alembic package
@thecaffiend
changed 'alembic' import to 'migrations'. typo fix
7827a3d
@thecaffiend
changed script_location to reflect change of 'alembic'->'migrations'
632f55e
@thecaffiend
added TODO about needing to change the db_url configuration when we s…
c54988a 
…tart using the orm models in things like api lambdas (where alembic isn't involved)
@thecaffiend
import order and logger changes.
cad38fa
@thecaffiend
cleaned up app.py and added a warning to it.
04285c9
@thecaffiend
handled case where we're not using alembic stuff and the config is no…
15a07ef 
…t valid.
@thecaffiend
updated with better doc for capedb and capedb-app scripts, as well as…
e660f6d 
… with sql doc
@thecaffiend
added note about requiring the database to exist to apply migrations …
37b6824 
…or create tables
@thecaffiend thecaffiend linked an issue Mar 5, 2026 that may be closed by this pull request
fix location and pathing to the capedb script and alembic files #9
Closed
@thecaffiend thecaffiend requested a review from mehalter March 5, 2026 19:26
mehalter
mehalter approved these changes Mar 5, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@mehalter mehalter left a comment

Choose a reason for hiding this comment

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

:shipit:

@thecaffiend thecaffiend merged commit c3bad64 into main Mar 5, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mehalter mehalter mehalter approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

fix location and pathing to the capedb script and alembic files

2 participants

@thecaffiend @mehalter