Skip to content

Stop calling setup.py in release CI#8771

Merged
aardappel merged 2 commits intogoogle:masterfrom
jtdavis777:fix/pep_517
Nov 17, 2025
Merged

Stop calling setup.py in release CI#8771
aardappel merged 2 commits intogoogle:masterfrom
jtdavis777:fix/pep_517

Conversation

@jtdavis777
Copy link
Collaborator

Fixes #8731

I am simply providing a PR which addresses the time sensitive concerns brought up by @EpicWink in the above issue :)

I have gitignored the LICENSE file in the project for now.

@github-actions github-actions bot added the CI Continuous Integration label Nov 11, 2025
@jtdavis777
Copy link
Collaborator Author

@dbaileychess and @aardappel the linked issue seems to indicate we've missed the deadline for changing this - figured I would get a PR up to handle the most pressing portion of the issue.

@aardappel aardappel merged commit 88b033b into google:master Nov 17, 2025
49 checks passed
@fliiiix
Copy link
Contributor

fliiiix commented Nov 18, 2025

TIL python3 -m build . just works for traditional setups, I guess when we drop Python 2 support we can modernize this setup

@jtdavis777 jtdavis777 deleted the fix/pep_517 branch November 19, 2025 00:13
TJKoury pushed a commit to DigitalArsenal/flatbuffers that referenced this pull request Nov 19, 2025
TJKoury pushed a commit to DigitalArsenal/flatbuffers that referenced this pull request Nov 19, 2025
TJKoury pushed a commit to DigitalArsenal/flatbuffers that referenced this pull request Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build Python with PEP 517

3 participants