Skip to content

Conversation

nabobalis
Copy link
Member

@nabobalis nabobalis commented Sep 10, 2025

Trying to make irispy nitpick proof.
Already patched sunraster.

@nabobalis nabobalis added the backport 2.3 on-merge: backport to 2.3 label Sep 10, 2025
@nabobalis nabobalis force-pushed the docs_nitpick branch 2 times, most recently from 9edd6a4 to 1ea7a30 Compare September 10, 2025 00:17
@nabobalis nabobalis added this to the 2.4.0 milestone Sep 10, 2025
@nabobalis nabobalis force-pushed the docs_nitpick branch 2 times, most recently from e384181 to 2bbf990 Compare September 10, 2025 00:34
@nabobalis nabobalis added No Changelog Entry Needed Skip all changelog checks. Minor Change PR only needs one approval to merge. labels Sep 10, 2025
@nabobalis
Copy link
Member Author

Need to propagate the nitpick file to the package template.

@wtbarnes wtbarnes changed the title I hate this package Nitpick documentation Sep 10, 2025

# -- Other options ----------------------------------------------------------

napoleon_use_rtype = False
Copy link
Member

Choose a reason for hiding this comment

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

Are these in the template? Why did you remove them?

Copy link
Member Author

Choose a reason for hiding this comment

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

None of this is in the template.

@Cadair Cadair merged commit ad5e8c9 into main Oct 2, 2025
19 checks passed
@Cadair Cadair deleted the docs_nitpick branch October 2, 2025 09:01
meeseeksmachine pushed a commit to meeseeksmachine/ndcube that referenced this pull request Oct 2, 2025
Cadair added a commit that referenced this pull request Oct 2, 2025
…on-2.3

Backport PR #881 on branch 2.3 (Nitpick documentation)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 2.3 on-merge: backport to 2.3 Minor Change PR only needs one approval to merge. No Changelog Entry Needed Skip all changelog checks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants