Skip to content

Add more resources and studies to flaky tests page in docs - #13250

Merged
nicoddemus merged 3 commits into
pytest-dev:mainfrom
gewenyu99:main
Mar 1, 2025
Merged

Add more resources and studies to flaky tests page in docs#13250
nicoddemus merged 3 commits into
pytest-dev:mainfrom
gewenyu99:main

Conversation

@gewenyu99

Copy link
Copy Markdown
Contributor

Hi there.

I found the flaky tests page in the Pytest docs, and the studies/resources here were helpful, some of which I've not seen.

I wanted to return the favor and share some newer resources and studies that I've found useful while building my solutions.

@gewenyu99

Copy link
Copy Markdown
Contributor Author

Two questions:

  • for a PR like this, should I create and issue first and attach change logs?
  • I have some other studies that aren't publically available in PDF form (as in you need access from the journal publishers), would it be helpful to include them still? Some of them are quite good.

@nicoddemus

nicoddemus commented Feb 24, 2025

Copy link
Copy Markdown
Member

Thanks! 🙇

for a PR like this, should I create and issue first and attach change logs?

No need, but if you want, you can add a 13250.doc.rst file to CHANGELOG -- but small changes like this are probably not worth it.

I have some other studies that aren't publically available in PDF form (as in you need access from the journal publishers), would it be helpful to include them still? Some of them are quite good.

I think it is fine, thanks! Perhaps add a *(requires account at X)* or *(paid access)* (something like that) to the end to warn readers.

Comment thread doc/en/explanation/flaky.rst
Comment thread doc/en/explanation/flaky.rst Outdated
@gewenyu99
gewenyu99 requested a review from nicoddemus February 27, 2025 20:15
@gewenyu99

Copy link
Copy Markdown
Contributor Author

Hey, I've fixed up the duplicate link. I can access the Divaportal link fine; it might be a case where the paper restricts access by region, but I can't seem to pinpoint it.

@webknjaz webknjaz added the skip news used on prs to opt out of the changelog requirement label Feb 28, 2025
Comment thread doc/en/explanation/flaky.rst Outdated
@webknjaz

Copy link
Copy Markdown
Member

@nicoddemus as many of these are whitepapers, would it make sense to integrate sphinxcontrib-bibtex?

@webknjaz webknjaz added the type: docs documentation improvement, missing or needing clarification label Feb 28, 2025
@nicoddemus

Copy link
Copy Markdown
Member

@nicoddemus as many of these are whitepapers, would it make sense to integrate sphinxcontrib-bibtex?

Sounds interesting for sure, but that would better be done in a separate PR I think. 👍

Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>

@nicoddemus nicoddemus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @gewenyu99!

@nicoddemus
nicoddemus enabled auto-merge (squash) March 1, 2025 13:35
@nicoddemus
nicoddemus merged commit 9f3c4d1 into pytest-dev:main Mar 1, 2025
@patchback

patchback Bot commented Mar 1, 2025

Copy link
Copy Markdown

Backport to 8.3.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.3.x/9f3c4d11d9b162d0926e9a3ae57ec6f262362d73/pr-13250

Backported as #13259

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback Bot pushed a commit that referenced this pull request Mar 1, 2025
---
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Bruno Oliveira <bruno@soliv.dev>

(cherry picked from commit 9f3c4d1)
nicoddemus pushed a commit that referenced this pull request Mar 1, 2025
…13259)

---
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Bruno Oliveira <bruno@soliv.dev>

(cherry picked from commit 9f3c4d1)

Co-authored-by: Vincent (Wen Yu) Ge <gewenyu99@gmail.com>
@gewenyu99

Copy link
Copy Markdown
Contributor Author

Great, thanks for reviewing these. @nicoddemus and @webknjaz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news used on prs to opt out of the changelog requirement type: docs documentation improvement, missing or needing clarification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants