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

Test out sphinx-comment with hypothesis.is to enable comments #897

Merged
merged 2 commits into from
Aug 18, 2022

Conversation

hunterhector
Copy link
Member

@hunterhector hunterhector commented Aug 18, 2022

This PR allows commenting on the documentation via hypothesis.is. This will also help commenting on the PRs

Description of changes

Added sphinx-comment package to doc build and updated docs/requirement.txt to include the package.

Possible influences of this PR.

The readthedoc site will not receive comments.

Test Conducted

Describe what test cases are included for the PR.

@codecov
Copy link

codecov bot commented Aug 18, 2022

Codecov Report

Merging #897 (4e7737a) into master (cb51c07) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #897   +/-   ##
=======================================
  Coverage   80.91%   80.91%           
=======================================
  Files         254      254           
  Lines       19551    19551           
=======================================
  Hits        15819    15819           
  Misses       3732     3732           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@hunterhector hunterhector merged commit d6b137d into asyml:master Aug 18, 2022
@hunterhector hunterhector deleted the overlay branch August 18, 2022 19:23
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.

1 participant