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

breaking: require Python 3.8, from Python 3.6 #1610

Merged
merged 2 commits into from
Jan 4, 2023

Conversation

consideRatio
Copy link
Member

  • breaking: require Python 3.8, from Python 3.6
  • pre-commit: run with pyupgrade --py38-plus

@consideRatio consideRatio added maintenance Under the hood improvements and fixes breaking labels Jan 4, 2023
@manics
Copy link
Member

manics commented Jan 4, 2023

Do you mind adding this breaking change to https://github.com/jupyterhub/binderhub/blob/main/CHANGES.md#breaking-changes ?

@manics
Copy link
Member

manics commented Jan 4, 2023

On second thoughts, let's leave the breaking changes doc till after all these breaking PRs are merged to avoid dealing with conflicts.... just need to remember 😄 (edit: didn't realise pasting a URL to a label displays the label in-line!)

Copy link
Member

@manics manics left a comment

Choose a reason for hiding this comment

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

Go ahead and merge if you're happy with the CI

@consideRatio
Copy link
Member Author

On second thoughts, let's leave the breaking changes doc till after all these breaking PRs are merged to avoid dealing with conflicts...

Yepp I was thinking the same thing!

Thanks for your review!!

@ofek
Copy link

ofek commented Jan 11, 2023

Hello! Just FYI now that 3.6 is no longer supported Hatchling can be used as the build backend. cc @blink1073 to confirm

@blink1073
Copy link

Yes, that is true, using hatch-jupyter-builder's migration script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking maintenance Under the hood improvements and fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants