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

refactor: use Enum for verbosity #66

Merged
merged 3 commits into from
Aug 10, 2023
Merged

refactor: use Enum for verbosity #66

merged 3 commits into from
Aug 10, 2023

Conversation

lukany
Copy link
Contributor

@lukany lukany commented Aug 8, 2023

Closes #65

@lukany lukany self-assigned this Aug 8, 2023
@lukany lukany changed the title Refactor/use enum refactor: use Enum for verbosity Aug 9, 2023
@lukany lukany force-pushed the refactor/use-enum branch 2 times, most recently from d50b3cb to d85be60 Compare August 9, 2023 07:19
@lukany lukany marked this pull request as ready for review August 9, 2023 07:19
@lukany lukany requested a review from mbelak-dtml August 9, 2023 08:38
edvart/report_sections/section_base.py Outdated Show resolved Hide resolved
edvart/report_sections/section_base.py Outdated Show resolved Hide resolved
@lukany lukany requested a review from mbelak-dtml August 9, 2023 13:24
Copy link
Collaborator

@mbelak-dtml mbelak-dtml left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge conflict

@mbelak-dtml mbelak-dtml added this pull request to the merge queue Aug 10, 2023
Merged via the queue into main with commit 521ce53 Aug 10, 2023
5 checks passed
@mbelak-dtml mbelak-dtml deleted the refactor/use-enum branch August 10, 2023 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Verbosity can be enum
2 participants