Skip to content

chore: ignore python build artifacts for library_generation tool#13258

Open
zhumin8 wants to merge 1 commit into
mainfrom
ignore-build
Open

chore: ignore python build artifacts for library_generation tool#13258
zhumin8 wants to merge 1 commit into
mainfrom
ignore-build

Conversation

@zhumin8
Copy link
Copy Markdown
Contributor

@zhumin8 zhumin8 commented May 22, 2026

ignore python build artifacts for library_generation tool

@zhumin8 zhumin8 marked this pull request as ready for review May 22, 2026 20:21
@zhumin8 zhumin8 requested a review from a team as a code owner May 22, 2026 20:21
@zhumin8 zhumin8 requested a review from suztomo May 22, 2026 20:21
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the .gitignore file to exclude Python build artifacts for the library_generation tool. Feedback suggests expanding the ignore list to include dist/ and *.egg-info/ directories to ensure comprehensive coverage of standard Python packaging artifacts.

Comment thread .gitignore
Comment on lines +90 to +91
# Python build artifacts for library_generation tool
sdk-platform-java/hermetic_build/library_generation/build/
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The PR aims to ignore Python build artifacts, but currently only specifies the build/ directory. Standard Python packaging tools (such as setuptools) also typically generate dist/ and *.egg-info/ directories during the build process. Including these will provide more comprehensive coverage and prevent accidental commits of these artifacts, consistent with the PR's objective.

# Python build artifacts for library_generation tool
sdk-platform-java/hermetic_build/library_generation/build/
sdk-platform-java/hermetic_build/library_generation/dist/
sdk-platform-java/hermetic_build/library_generation/*.egg-info/

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants