Skip to content

Modernise and align API with FastAPI idiomatic patterns#22

Merged
THEGOLDENPRO merged 16 commits into
mainfrom
refactor/modernisation-and-cleanup
May 3, 2026
Merged

Modernise and align API with FastAPI idiomatic patterns#22
THEGOLDENPRO merged 16 commits into
mainfrom
refactor/modernisation-and-cleanup

Conversation

@THEGOLDENPRO
Copy link
Copy Markdown
Owner

@THEGOLDENPRO THEGOLDENPRO commented Apr 19, 2026

  • Switched to UV for the docker image and added a lock file so code is more reproducible.
  • Dropped all past Python versions to prioritize for Python 3.12+ support (the docker image also uses 3.12).
  • Replaced unmaintained slowapi library with fastapi-limiter for rate limiting (this was a blocker while making the code more idiomatic).
  • Removed GIT_REPO_x environment variables as it makes no sense to even use them.
  • Updated to the latest dependencies (including fastapi).
  • Brought in some new tooling (uv and ty) to improve maintainability.
  • Allowed passing proxy headers to the API to help with rate limiting.
  • Cleaned up and removed a bunch of my old python code (not everything has been refactored).
  • Removed devgoldyutils and typing-extensions.
  • Updated some README instructions.
  • Added ty and ruff check workflows.

…eaders in docker, switch to pydantic models, remove devgoldyutils and typing extensions and modernize and clean up everything else (quite big I know)
…onstants, move books api section to separate router and make programming books class a dependency
@THEGOLDENPRO THEGOLDENPRO self-assigned this Apr 19, 2026
@THEGOLDENPRO THEGOLDENPRO added the enhancement New feature or request label Apr 19, 2026
@THEGOLDENPRO
Copy link
Copy Markdown
Owner Author

THEGOLDENPRO commented Apr 19, 2026

I've done a lot of changes so I'll need to test run this for a while before I push it to docker hub. In theory nothing should have changed from the client facing side (all api wrappers and client should just work). I didn't touch any responses on purpose.

I'm new to using uv in docker and the fastapi-limiter library so please do let me know if anything there or anywhere else in fact is out of place.

@THEGOLDENPRO
Copy link
Copy Markdown
Owner Author

Should merge this soon.

@THEGOLDENPRO
Copy link
Copy Markdown
Owner Author

Okay merging this now, I'll do a github and docker release later.

@THEGOLDENPRO THEGOLDENPRO merged commit 2cb906b into main May 3, 2026
3 checks passed
@THEGOLDENPRO THEGOLDENPRO deleted the refactor/modernisation-and-cleanup branch May 3, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant