Skip to content

Fix Read the Docs build environment#887

Merged
kenhoberkeley merged 2 commits intoucb-bar:masterfrom
daniellovell:fix-rtd-build
Mar 1, 2025
Merged

Fix Read the Docs build environment#887
kenhoberkeley merged 2 commits intoucb-bar:masterfrom
daniellovell:fix-rtd-build

Conversation

@daniellovell
Copy link
Collaborator

@daniellovell daniellovell commented Feb 28, 2025

Pin the poetry version to 1.5.1, which is known to work well with ReadTheDocs Explicitly install virtualenv 20.23.1, which doesn't have the issue with the 'free_threaded' attribute

Related PRs / Issues
Ideally fixes the failing check in #886
(tangential to the contents of that PR)

Type of change:

  • Bug fix
  • New feature
  • Other enhancement

Impact:

  • Change to core Hammer
  • Change to a Hammer plugin
  • Other

Contributor Checklist:

  • Did you set master as the base branch?
  • Did you state the type-of-change/impact?
  • Did you delete any extraneous prints/debugging code?
  • (If applicable) Did you add documentation for the feature?
  • (If applicable) Did you update the poetry.lock file if you updated the requirements in pyproject.toml?
  • (If applicable) Did you add a unit test demonstrating the PR?
  • (If applicable) Did you run this through the e2e integration tests?
  • (If applicable) Did you update the submodules in e2e/ if this feature depends on updated plugins?

Pin the poetry version to 1.4.2, which is known to work well with ReadTheDocs
Explicitly install virtualenv 20.23.1, which doesn't have the issue with the 'free_threaded' attribute
@daniellovell daniellovell changed the title Fix ReadTheDocs build environment Fix Read the Docs build environment Feb 28, 2025
Poetry 1.5.1 should support dependency groups (spamming the autograder)
@kenhoberkeley kenhoberkeley merged commit 461e7e7 into ucb-bar:master Mar 1, 2025
4 checks passed
@daniellovell daniellovell deleted the fix-rtd-build branch March 1, 2025 00:54
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.

2 participants