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

⬆️ Allow more recent version of packaging #1196

Merged
merged 1 commit into from Feb 21, 2023

Conversation

r-richmond
Copy link
Contributor

Why make change

The current specification conflicts with new versions of black
https://github.com/psf/black/blob/d9b8a6407e2f46304a8d36b18e4a73d8e0613519/pyproject.toml#L68

What do the new versions of packaging do

https://github.com/pypa/packaging/releases/tag/22.0
of note

  1. Removes support for python 3.6 (which this package doesn't support already)
  2. Adds support for python 3.11 (Which this isn't marked as supporting yet but probably should at some point)

@salesforce-cla
Copy link

Thanks for the contribution! Before we can merge this, we need @r-richmond to sign the Salesforce.com Contributor License Agreement.

@r-richmond
Copy link
Contributor Author

CLA signed previously as noted here

@jacalata
Copy link
Contributor

hm, looks like that brings in some changes from black as well. Can you run black to get the code back into a valid state? It looks like the vast majority of changes are this one:
Remove trailing newlines after code block open (psf/black#3035) (22.6.0)

Also, could you bump the named version to 22.0, for clarity?

@r-richmond
Copy link
Contributor Author

Addressed both black run & bumping minimum version as requested (also added comment for more clarity)

@jacalata jacalata merged commit ccdd790 into tableau:master Feb 21, 2023
@r-richmond r-richmond deleted the patch-1 branch February 21, 2023 00:35
@r-richmond
Copy link
Contributor Author

@jacalata do you have a rough estimate for when the next version will go out?

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

Successfully merging this pull request may close these issues.

None yet

2 participants