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

docs: fix #104 #111

Merged
merged 3 commits into from
Jan 15, 2025
Merged

docs: fix #104 #111

merged 3 commits into from
Jan 15, 2025

Conversation

nh13
Copy link
Member

@nh13 nh13 commented Jan 15, 2025

Fixed #104

@nh13 nh13 requested a review from tfenne as a code owner January 15, 2025 05:47
Base automatically changed from tf_reorg_phase1 to dev January 15, 2025 12:38
@tfenne
Copy link
Member

tfenne commented Jan 15, 2025

@nh13 Sorry - I didn't see this before I fixed up my PR and merged it to dev. Would you mind rebasing please?

Copy link
Member

@tfenne tfenne left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this. I'd seen examples online where the triple-backticks weren't necessary, but I wonder if that's just wrong or a sphinx vs. mkdocs thing.

Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.32%. Comparing base (a40056b) to head (bcccc06).
Report is 1 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #111   +/-   ##
=======================================
  Coverage   97.32%   97.32%           
=======================================
  Files          21       21           
  Lines        1642     1642           
  Branches      195      195           
=======================================
  Hits         1598     1598           
  Misses         26       26           
  Partials       18       18           

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

@nh13 nh13 merged commit cba8c74 into dev Jan 15, 2025
5 checks passed
@nh13 nh13 deleted the fix/104 branch January 15, 2025 16:15
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