Skip to content

Refactor the application#39

Draft
gnramsay wants to merge 16 commits into
mainfrom
refactoring
Draft

Refactor the application#39
gnramsay wants to merge 16 commits into
mainfrom
refactoring

Conversation

@gnramsay

@gnramsay gnramsay commented Sep 3, 2023

Copy link
Copy Markdown
Owner

Refactor and re-arrange the app.

This project was started a while ago when my Python knowledge was lesser, so now I should be able to include more best practices. Also, I can clean the code up a good bit.

  • Move to uv from poetry
  • Convert the main library code into a module
  • Add typing
  • Move static class methods into a helper file instead

Signed-off-by: Grant Ramsay <grant@gnramsay.com>
@gnramsay gnramsay self-assigned this Sep 3, 2023
gnramsay and others added 15 commits September 3, 2023 17:58
Signed-off-by: Grant Ramsay <grant@gnramsay.com>
Signed-off-by: Grant Ramsay <grant@gnramsay.com>
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
it expects an extra CR which is a pain for auto-generated files without it.

Signed-off-by: Grant Ramsay <seapagan@gmail.com>
just to keep my local checkout cleaner

Signed-off-by: Grant Ramsay <seapagan@gmail.com>
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
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.

2 participants