Skip to content

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
Co-authored-by: Wouter van Oortmerssen <aardappel@gmail.com>
TJKoury pushed a commit to DigitalArsenal/flatbuffers that referenced this pull request Nov 19, 2025
Co-authored-by: Wouter van Oortmerssen <aardappel@gmail.com>
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