Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Bump the Tutor version dependency to >=16.1.7 #27

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

fghaas
Copy link
Contributor

@fghaas fghaas commented Nov 17, 2023

We rely on the Tutor OPENEDX_COMMON_VERSION variable to determine the tag (or branch) to fetch for the Tubular repository.

In Tutor 16.1.6, the default value for OPENEDX_COMMON_VERSION changed from open-release/palm.3 to open-release/palm.4. This means we now need a version bump in our own install_requires list, so that we require Tutor>=16.1.6.

However, the Tutor 16.1.6 package as published to PyPI contains a bug that makes it uninstallable, so we have to require Tutor>=16.1.7 instead.

References:
https://github.com/overhangio/tutor/releases/tag/v16.1.6
overhangio/tutor#943

We rely on the Tutor OPENEDX_COMMON_VERSION variable to determine
the tag (or branch) to fetch for the Tubular repository.

In Tutor 16.1.6, the default value for OPENEDX_COMMON_VERSION changed
from open-release/palm.3 to open-release/palm.4. This means we now
need a version bump in our own install_requires list, so that we
require Tutor>=16.1.6.

However, the Tutor 16.1.6 package as published to PyPI contains a bug
that makes it uninstallable, so we have to require Tutor>=16.1.7
instead.

References:
https://github.com/overhangio/tutor/releases/tag/v16.1.6
overhangio/tutor#943
@fghaas
Copy link
Contributor Author

fghaas commented Nov 20, 2023

@mrtmm With Tutor 16.1.7 having been released this no longer breaks, so this is good for review now.

@fghaas fghaas changed the title fix: Bump the Tutor version dependency to >=16.1.6 fix: Bump the Tutor version dependency to >=16.1.7 Nov 20, 2023
@fghaas fghaas merged commit 2620959 into hastexo:main Nov 20, 2023
4 checks passed
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