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

feat: Add HTML and excel depth report #18

Merged
merged 4 commits into from
Jan 22, 2024
Merged

feat: Add HTML and excel depth report #18

merged 4 commits into from
Jan 22, 2024

Conversation

chels0
Copy link

@chels0 chels0 commented Jan 19, 2024

This PR:

(If this is a release PR, no need to add following. Leave this part empty)
(Use the following lines to create a PR text body. Make sure to remove all non-relevant one after you're done)
(Repeat each field as many times as necessary)

Added: for new features.
Changed: for changes in existing functionality.
Deprecated: for soon-to-be removed features.
Removed: for now removed features.
Fixed: for any bug fixes.
Security: in case of vulnerabilities.

Review and tests:

  • Tests pass
  • If you've fixed a bug or added code that should be tested, add tests!
  • Code review
  • CHANGELOG.md is updated
  • New code is executed and covered by tests, and test approve
  • Module compatibility section in README.md is up to date

@chels0 chels0 requested review from pyrevo and ljmesi as code owners January 19, 2024 10:59
@chels0 chels0 changed the title Depth report Feat: Add HTML and excel depth report Jan 19, 2024
@chels0 chels0 changed the title Feat: Add HTML and excel depth report feat: Add HTML and excel depth report Jan 19, 2024
Copy link
Member

@pyrevo pyrevo left a comment

Choose a reason for hiding this comment

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

It looks good to me!

@chels0 chels0 merged commit 777e1c0 into develop Jan 22, 2024
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants