This repository has been archived by the owner on Nov 16, 2024. It is now read-only.
First round of feedback from CRSE
Closed Jun 28, 2022
100% complete
I had posted the code up to commit 6020a4f to the Code Review Stack Exchange (abbreviated CRSE). Good feedback has been received, and will be acted up on.
- Post 1 of 3, Core: https://codereview.stackexchange.com/q/276225/257004
- Post 2 of 3, CLI: https://codereview.stackexchange.com/q/276227/257004
- Post 3 of 3, Database: https://codereview.stackexchange.co…
I had posted the code up to commit 6020a4f to the Code Review Stack Exchange (abbreviated CRSE). Good feedback has been received, and will be acted up on.
- Post 1 of 3, Core: https://codereview.stackexchange.com/q/276225/257004
- Post 2 of 3, CLI: https://codereview.stackexchange.com/q/276227/257004
- Post 3 of 3, Database: https://codereview.stackexchange.com/q/276229/257004
I had also posted up to commit 3250a1a to r/learnpython on Reddit, and got some good feedback there too.
- Post: https://www.reddit.com/r/learnpython/comments/uewhvl/please_help_me_review_a_cli_application_that_ive/
- Feedback: https://www.reddit.com/r/learnpython/comments/uevol4/comment/i6pr78v/?context=3
Each comment tackles one file/aspect of the upgrades. The pull request immedeatly following a comment will have a summary of the changes and decisions made, with respect to the issue comment.