Skip to content

Make spelling test author-only - #59

Open
salva wants to merge 1 commit into
masterfrom
fix/author-spelling-test
Open

Make spelling test author-only#59
salva wants to merge 1 commit into
masterfrom
fix/author-spelling-test

Conversation

@salva

@salva salva commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Summary

Make the spelling test author-only and add the missing stopwords observed during review.

Changes

  • Skip t/pod-spell.t unless AUTHOR_TESTING is set.
  • Add stopwords for X11, didn, and doesn.
  • Add the tracked spelling test to MANIFEST.

Fixes #52.

Testing

  • perl -Ilib t/pod-spell.t
  • AUTHOR_TESTING=1 perl -Ilib t/pod-spell.t

Copilot AI review requested due to automatic review settings June 4, 2026 11:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces environment-related flakiness in the distribution’s default test run by making the POD spelling check an author-only test, while also updating the stopword list to match words observed failing in real environments.

Changes:

  • Gate t/pod-spell.t behind AUTHOR_TESTING to avoid running it in normal prove/CPAN tester runs.
  • Add stopwords for X11, didn, and doesn to address the known failures.
  • Add t/pod-spell.t to MANIFEST so it’s tracked in the release file list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
t/pod-spell.t Skips unless AUTHOR_TESTING is set; expands stopwords to cover observed failures.
MANIFEST Adds t/pod-spell.t to the distribution manifest.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

Spelling test is environment-sensitive and fails with current stopwords

2 participants