MAINT: Options to catch more issues reported by pytest by DimitriPapadopoulos · Pull Request #28983 · numpy/numpy · GitHub
Skip to content

MAINT: Options to catch more issues reported by pytest - #28983

Merged
seberg merged 1 commit into
numpy:mainfrom
DimitriPapadopoulos:pytest
Jul 14, 2025
Merged

seberg merged 1 commit into
numpy:mainfrom
DimitriPapadopoulos:pytest

Conversation

@DimitriPapadopoulos

@DimitriPapadopoulos DimitriPapadopoulos commented May 16, 2025

Copy link
Copy Markdown
Contributor

Apply Scientific Python Development Guide suggestions PP306, PP307, PP308. Skip PP305 for now.

@DimitriPapadopoulos

DimitriPapadopoulos commented May 16, 2025

Copy link
Copy Markdown
Contributor Author

@seberg

seberg commented Jul 9, 2025

Copy link
Copy Markdown
Member

Not sure how to retrieve detailed logs to fix these issues:

Just scroll up? It may be very hard to get xfail strict to work and I am not sure we should ship that, because we won't test all weird platforms.

Apply Repo-Review suggestions PP306, PP307, PP308. Skip PP305 for now.
@DimitriPapadopoulos
DimitriPapadopoulos marked this pull request as ready for review July 12, 2025 12:38
@seberg seberg changed the title MNT: Options to catch more issues reported by pytest MAINT: Options to catch more issues reported by pytest Jul 14, 2025
@seberg

seberg commented Jul 14, 2025

Copy link
Copy Markdown
Member

@seberg
seberg merged commit 9173e54 into numpy:main Jul 14, 2025
@DimitriPapadopoulos
DimitriPapadopoulos deleted the pytest branch July 14, 2025 14:53
@DimitriPapadopoulos
DimitriPapadopoulos restored the pytest branch July 14, 2025 14:53
@DimitriPapadopoulos
DimitriPapadopoulos deleted the pytest branch July 14, 2025 14:54
mattip added a commit that referenced this pull request Oct 11, 2025
DEP: finalize deprecation of numpy/fft.helpers and numpy.linalg.linalg (closes #28983)
IndifferentArea pushed a commit to IndifferentArea/numpy that referenced this pull request Dec 7, 2025
Apply Repo-Review suggestions PP306, PP307, PP308. Skip PP305 for now.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants