Repo maintenance: remove un-needed compiled Python byte code and update gitignore to prevent accidental commiting of temp files - #445
Merged
Conversation
This commit updates the gitignore and also removes the .pyc files already committed to the repository. The updates to the .gitignore consist of an explicit ignore for the `documentation/extras/extensions` directory and content recommended for Python projects from: https://www.toptal.com/developers/gitignore/api/python This content should not be included in the repository because it comprises of dynamically generated python byte code, and the content in documentation/extras/extensions is also dynamically added at run time and is not governed as part of the Standard. Some .pyc files were already added to the project erroneously, so this commit removes them. They are artifacts from the build process and not content for the Standard or the Docs. The updates to the .gitignore prevents these files from being committed to the repository by accident in the future.
Contributor
|
@mrshll1001 Agreed this does not trigger a PATCH or require approval from 360Giving staff or Stewardship Committee |
BibianaC
approved these changes
Jul 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the gitignore and also removes the .pyc files
already committed to the repository.
The updates to the .gitignore consist of an explicit ignore for
the
documentation/extras/extensionsdirectory and content recommendedfor Python projects from:
https://www.toptal.com/developers/gitignore/api/python
This content should not be included in the repository because it
comprises of dynamically generated python byte code, and the content in
documentation/extras/extensions is also dynamically added at run time
and is not governed as part of the Standard.
Some .pyc files were already added to the project erroneously, so this
commit removes them. They are artifacts from the build process and not
content for the Standard or the Docs.
The updates to the .gitignore prevents these files from being committed
to the repository by accident in the future.
This makes Standard maintenance cleaner and easier going forward.
Since this is internal repository maintenance and doesn't touch the Standard or the Docs content, I don't think this requires approval by the SC or PATCH process, however it'd be good if someone from the 360Giving team could give this approval.