Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

adds .tox to ignore path #22

Merged
merged 2 commits into from
Mar 18, 2023
Merged

adds .tox to ignore path #22

merged 2 commits into from
Mar 18, 2023

Conversation

zackees
Copy link
Contributor

@zackees zackees commented Mar 17, 2023

Addresses:

#21

@zackees zackees requested a review from mpoon as a code owner March 17, 2023 05:34
@zackees
Copy link
Contributor Author

zackees commented Mar 17, 2023 •
edited
Loading

Added fix for win32 and also additional python binary files and mypy spam that should be ignored.

Also the if a the target repo doesn't include a gpt ignore file then the default from the source repo is used.

mpoon
mpoon approved these changes Mar 18, 2023
View reviewed changes
Copy link
Owner

@mpoon mpoon left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@mpoon mpoon merged commit bc4b363 into mpoon:main Mar 18, 2023
@zackees zackees deleted the ignore-tox branch March 19, 2023 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mpoon mpoon mpoon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zackees @mpoon