Skip to content

Add docstring tests - #7219

Merged
cknitt merged 1 commit into
masterfrom
doc-tests-fix
Jan 5, 2025
Merged

Add docstring tests#7219
cknitt merged 1 commit into
masterfrom
doc-tests-fix

Conversation

@cknitt

@cknittcknitt commented Dec 27, 2024•
edited
Loading

Copy link
Copy Markdown
Member
  • Add Pervasives.assertEqual
  • Add assertions to doc examples
  • Add docstring tests and run them in CI:
    • Extract docstrings into Mocha tests (in a single source file for performance)
    • Compile and run Mocha tests

Supersedes #7184

@cknittcknitt changed the title Attempt to fix doc tests for macOSAdd docstring testsDec 30, 2024
@cknitt
cknittforce-pushed the doc-tests-fix branch 2 times, most recently from bf0224c to edce512CompareJanuary 4, 2025 08:30
@cknitt
cknitt marked this pull request as ready for review January 4, 2025 08:30
@cknitt
cknitt requested a review from zthJanuary 4, 2025 08:34
@cknitt@aspeddro
Add docstring tests
b5fb5be
Co-authored-by: Pedro Castro <aspeddro@gmail.com>
@cknitt
cknitt merged commit 3500728 into masterJan 5, 2025
@cknitt
cknitt deleted the doc-tests-fix branch January 5, 2025 07:43
@cknittcknitt mentioned this pull request Jan 5, 2025
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fhammerschmidtfhammerschmidtfhammerschmidt approved these changes
@zthzthAwaiting requested review from zth

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cknitt@fhammerschmidt