Skip to content

Python 3.13 support#1002

Merged
bdraco merged 4 commits intomasterfrom
py313
Sep 9, 2024
Merged

Python 3.13 support#1002
bdraco merged 4 commits intomasterfrom
py313

Conversation

@bdraco
Copy link
Copy Markdown
Member

@bdraco bdraco commented Sep 9, 2024

needs #993 to pass

What do these changes do?

Support Python 3.13 along with wheels

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Sep 9, 2024
Comment thread .github/workflows/ci-cd.yml Outdated
@bdraco bdraco marked this pull request as ready for review September 9, 2024 22:37
@bdraco bdraco merged commit f3876fd into master Sep 9, 2024
@bdraco bdraco deleted the py313 branch September 9, 2024 22:38
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.pyver }}
allow-prereleases: >-
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@bdraco this should be kept long-term so that adding -dev things in the future wouldn't require remembering to reimplement it once again.

Copy link
Copy Markdown
Member Author

@bdraco bdraco Sep 9, 2024

Choose a reason for hiding this comment

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

I matched it to yarl since I wasn't sure if 3.13-dev would pull non rc/stable 3.13, and allow-prereleases: true is still needed for 3.13 since its rc

https://github.com/aio-libs/yarl/blob/eaf76414d44e72b022a7dee3b48bb532934a2f33/.github/workflows/ci-cd.yml#L226

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll revert that change

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Just realized there are other checks for -dev though that we want to be fatal if 3.13 breaks though....so I'm not sure if reverting it makes sense

Copy link
Copy Markdown
Member

@webknjaz webknjaz Sep 9, 2024

Choose a reason for hiding this comment

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

Okay, leave it as is. I usually like using a different syntax ~3.13.0-0 and check for ~ in other places. But I suppose it's not important to figure out right now. Something to remember at some point, perhaps.

I also didn't fully realize that this is about the action input. I typically have this in the check for allowing failures, which is what confused me and I overlooked the context.

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

Labels

bot:chronographer:provided There is a change note present in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants