Skip to content

Development: refresh outdated project fixtures#12834

Open
SergioChan wants to merge 4 commits intoreadthedocs:mainfrom
SergioChan:chore/update-dev-fixture-projects-8914
Open

Development: refresh outdated project fixtures#12834
SergioChan wants to merge 4 commits intoreadthedocs:mainfrom
SergioChan:chore/update-dev-fixture-projects-8914

Conversation

@SergioChan
Copy link

Summary

  • Replace six outdated/abandoned fixture repositories in readthedocs/projects/fixtures/test_data.json with currently maintained projects used by the community.
  • Keep fixture structure unchanged and limit edits to name, slug, and repo fields for those entries.
  • This updates local bootstrap data to better reflect active projects referenced in issue discussion.

Testing

  • python -m json.tool readthedocs/projects/fixtures/test_data.json >/dev/null

Related

@SergioChan SergioChan requested a review from a team as a code owner March 10, 2026 16:28
@SergioChan SergioChan requested a review from stsewd March 10, 2026 16:28
"slug": "poliastro",
"description": "Awesome",
"repo": "http://github.com/pinax/pinax",
"repo": "https://github.com/poliastro/poliastro",
Copy link
Member

Choose a reason for hiding this comment

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

Looks like this project has been archived. https://github.com/psf/requests could be a good replacement.

@SergioChan
Copy link
Author

Thanks for the catch — updated in fb233a5.

  • Replaced archived poliastro fixture entry with requests.
  • Updated fixture fields to:
    • name: requests
    • slug: requests
    • repo: https://github.com/psf/requests

Validation:

  • python -m json.tool readthedocs/projects/fixtures/test_data.json

Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

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

This adds a few huge docs with really complex build requirements -- maybe that's good for local testing, but we probably mostly want things that are easy to build without any downstream dependencies.

@SergioChan
Copy link
Author

SergioChan commented Mar 12, 2026

Thanks for the detailed review feedback

working through the requested updates and will post a focused patch (or direct answers) shortly.

@SergioChan
Copy link
Author

Thanks for the guidance — I’ve pushed a focused follow-up to reduce heavy fixture choices and keep this set easier to build.

Updated fixtures in readthedocs/projects/fixtures/test_data.json:

  • xarrayurllib3 (https://github.com/urllib3/urllib3)
  • geopandasflask (https://github.com/pallets/flask)
  • godotsphinx (https://github.com/sphinx-doc/sphinx)

This keeps the refresh intent while avoiding the larger downstream dependency profiles from the previous picks.

Validation:

  • python -m json.tool readthedocs/projects/fixtures/test_data.json

@SergioChan
Copy link
Author

Thanks for the earlier review — I pushed a focused follow-up commit to address the failing fixture state:

  • Commit: c2ab45a
  • Change: replaced the duplicate slug: "sphinx" fixture entry with a unique lightweight project entry:
    • name: "Jinja"
    • slug: "jinja"
    • repo: https://github.com/pallets/jinja

This keeps the fixture refresh scope while avoiding duplicate project slugs in test_data.json.

Validation run locally:

  • python -m json.tool readthedocs/projects/fixtures/test_data.json
  • duplicate-slug check on projects.project fixture entries (now no duplicates)

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.

Development: update initial fixture

3 participants