Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release 0.9.0 #52

Merged
merged 29 commits into from
Feb 14, 2022
Merged

Release 0.9.0 #52

merged 29 commits into from
Feb 14, 2022

Conversation

rszamszur
Copy link
Member

@rszamszur rszamszur commented Feb 13, 2022

Checklist:

  • Added tests for changed code where applicable.
  • Documentation reflects the changes where applicable.
  • Updated the CHANGELOG.md file with your changes.
  • My PR is ready to review.

Resolves: #48
Resolves: #50
Resolves: #51

Description of the changes being introduced by the pull request:
This is a quite big change, but mostly refactoring current codebase + fix for initial fastapi-mvc run command implementation.
Reasons and more info about refactor can be found in the issue. Lastly, fastapi-mvc.ini parser properties were extended.

@rszamszur rszamszur added bug Something isn't working enhancement New feature or request refactor Related to major code refactoring/reorganisation labels Feb 13, 2022
@rszamszur rszamszur added this to the 0.9.0 milestone Feb 13, 2022
@rszamszur rszamszur self-assigned this Feb 13, 2022
@rszamszur rszamszur mentioned this pull request Feb 13, 2022
8 tasks
@rszamszur rszamszur marked this pull request as ready for review February 14, 2022 20:39
@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2022

@codecov-commenter
Copy link

codecov-commenter commented Feb 14, 2022

Codecov Report

Merging #52 (a15cd83) into master (f7b7036) will increase coverage by 0.30%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
+ Coverage   99.19%   99.49%   +0.30%     
==========================================
  Files          16       34      +18     
  Lines         371      591     +220     
==========================================
+ Hits          368      588     +220     
  Misses          3        3              
Impacted Files Coverage Δ
fastapi_mvc/cli/cli.py 100.00% <100.00%> (ø)
fastapi_mvc/cli/new.py 100.00% <100.00%> (ø)
fastapi_mvc/cli/run.py 100.00% <100.00%> (ø)
fastapi_mvc/commands/__init__.py 100.00% <100.00%> (ø)
fastapi_mvc/commands/base.py 100.00% <100.00%> (ø)
fastapi_mvc/commands/install_project.py 100.00% <100.00%> (ø)
fastapi_mvc/commands/invoker.py 100.00% <100.00%> (ø)
fastapi_mvc/commands/new_project.py 100.00% <100.00%> (ø)
fastapi_mvc/commands/run_uvicorn.py 100.00% <100.00%> (ø)
fastapi_mvc/commands/verify_install.py 100.00% <100.00%> (ø)
... and 14 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 769c4ab...a15cd83. Read the comment docs.

@rszamszur rszamszur merged commit 0a5dc32 into master Feb 14, 2022
@rszamszur rszamszur deleted the r.0.9.0 branch February 14, 2022 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request refactor Related to major code refactoring/reorganisation
Projects
None yet
2 participants