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

HTML search: add test coverage, and introduce test module file suffix #12759

Conversation

jayaddison
Copy link
Contributor

Feature or Bugfix

  • Refactoring

Purpose

  • Add test coverage for search: order non-main index entries after other results #11696.
    • Index entries created using the :index: directive generally appear below similar matches in document section titles -- unless the exclamation mark (index entry priority / main flag) is set in the source reference.
  • Add test coverage for partial string search matching.
  • Use a .spec.js suffix for JavaScript test modules.

Detail

  • N/A

Relates

@AA-Turner AA-Turner merged commit 3bab324 into sphinx-doc:master Aug 11, 2024
21 checks passed
@jayaddison
Copy link
Contributor Author

Thanks!

@jayaddison jayaddison deleted the pr-11696-followup/search-indexentries-test-coverage branch August 11, 2024 14:19
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 11, 2024
@AA-Turner AA-Turner added this to the 8.1.x milestone Oct 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants