FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: correct typo in test comment by TheGreatApollyon · Pull Request #20177 · astropy/astropy · GitHub

fix: correct typo in test comment - #20177

Open
TheGreatApollyon wants to merge 1 commit into
astropy:mainfrom
TheGreatApollyon:fix/typos
Open

fix: correct typo in test comment#20177
TheGreatApollyon wants to merge 1 commit into
astropy:mainfrom
TheGreatApollyon:fix/typos

Conversation

Copy link
Copy Markdown

Fixes: encapuslates → encapsulates in test_nonstandard.py

Copilot AI review requested due to automatic review settings July 31, 2026 06:06
TheGreatApollyon requested a review from saimn as a code owner July 31, 2026 06:06

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.

  • Do the proposed changes actually accomplish desired goals?
  • Do the proposed changes follow the Astropy coding guidelines?
  • Are tests added/updated as required? If so, do they follow the Astropy testing guidelines?
  • Are docs added/updated as required? If so, do they follow the Astropy documentation guidelines?
  • Is rebase and/or squash necessary? If so, please provide the author with appropriate instructions. Also see instructions for rebase and squash.
  • Did the CI pass? If no, are the failures related? If you need to run daily and weekly cron jobs as part of the PR, please apply the "Extra CI" label. Codestyle issues can be fixed by the bot.
  • Is a change log needed? If yes, did the change log check pass? If no, add the "no-changelog-entry-needed" label. If this is a manual backport, use the "skip-changelog-checks" label unless special changelog handling is necessary.
  • Is this a big PR that makes a "What's new?" entry worthwhile and if so, is (1) a "what's new" entry included in this PR and (2) the "whatsnew-needed" label applied?
  • At the time of adding the milestone, if the milestone set requires a backport to release branch(es), apply the appropriate "backport-X.Y.x" label(s) before merge.

This comment was marked as spam.

pllim added this to the v7.2.3 milestone Jul 31, 2026
pllim added Docs Bug backport-v7.2.x on-merge: backport to v7.2.x backport-v8.0.x on-merge: backport to v8.0.x labels Jul 31, 2026

pllim commented Jul 31, 2026

Copy link
Copy Markdown
Member

That is a lot of energy and water wasted for this change. Unfortunate.

Are you a human?

Copy link
Copy Markdown
Author

That is a lot of energy and water wasted for this change. Unfortunate.

Are you a human?

As much as one can be ig

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v7.2.x on-merge: backport to v7.2.x backport-v8.0.x on-merge: backport to v8.0.x Bug Docs io.fits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL