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.

Sign up for GitHub

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

Jump to bottom

refactorings #10

Merged
merged 5 commits into from
Jan 28, 2022
Merged

refactorings #10

merged 5 commits into from
Jan 28, 2022

Conversation

Nicoretti
Copy link
Member

  • Code and configuration refactorings
  • Merge modules _cli.py and cli.py
  • Rename module _main.py to application.py
  • Change code coverrage limit
  • Add support for coveralls code coverage upload

@Nicoretti
Code and configuration refactorings
1b6e742
@Nicoretti
Merge modules _cli.py and cli.py
03add4d
@Nicoretti
Rename module _main.py to application.py
34aad1a
@Nicoretti
Change code coverrage limit
b08a136 
the previous code coverage of 100% was achieved by using "#pragma: nocover"
directives, which basically hides uncovered code paths. In the future
we want to keep the code coverage high but not at the expense of hiding
uncovered paths, therefore all "#pragmas" have been removed and
the lower limit was set to 97%.
@Nicoretti Nicoretti force-pushed the refactorings branch 4 times, most recently from f97bfe1 to 955a9a5 Compare January 28, 2022 12:33
@Nicoretti Nicoretti merged commit d1d882b into master Jan 28, 2022
@Nicoretti Nicoretti deleted the refactorings branch January 28, 2022 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Nicoretti