Skip to content

Comments

fix docs hyperlinks#133

Merged
omesser merged 3 commits intomainfrom
test-coverage
Apr 2, 2023
Merged

fix docs hyperlinks#133
omesser merged 3 commits intomainfrom
test-coverage

Conversation

@casperdcl
Copy link
Collaborator

@casperdcl casperdcl commented Apr 2, 2023 •
edited
Loading

  • add public typehints
  • docs: tidy docstrings
  • docs: fix hyperlinks

@casperdcl casperdcl self-assigned this Apr 2, 2023
@casperdcl casperdcl added documentation Markdown files testing Unit tests & debugging technical-debt Refactoring, linting & tidying labels Apr 2, 2023
@casperdcl casperdcl changed the title test coverage fix docs hyperlinks, increase test coverage Apr 2, 2023
@codecov-commenter
Copy link

codecov-commenter commented Apr 2, 2023 •
edited by codecov bot
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.39%. Comparing base (68ac169) to head (743dd7a).
⚠️ Report is 100 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   91.39%   91.39%           
=======================================
  Files           3        3           
  Lines         337      337           
=======================================
  Hits          308      308           
  Misses         29       29

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@casperdcl casperdcl changed the title fix docs hyperlinks, increase test coverage fix docs hyperlinks Apr 2, 2023
@casperdcl casperdcl removed the testing Unit tests & debugging label Apr 2, 2023
omesser
omesser approved these changes Apr 2, 2023
View reviewed changes
Copy link
Contributor

@omesser omesser left a comment

Choose a reason for hiding this comment

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

🙏 Thank you @casperdcl

@omesser omesser merged commit 80a1050 into main Apr 2, 2023
@omesser omesser deleted the test-coverage branch April 2, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@0x2b3bfa0 0x2b3bfa0 0x2b3bfa0 approved these changes

@shcheklein shcheklein Awaiting requested review from shcheklein

+1 more reviewer

@omesser omesser omesser approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@casperdcl casperdcl

Labels

documentation Markdown files technical-debt Refactoring, linting & tidying

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@casperdcl @codecov-commenter @0x2b3bfa0 @omesser