Skip to content
This repository has been archived by the owner on Mar 10, 2024. It is now read-only.

feat: add option to ignore untracked files #31

Merged
merged 3 commits into from
May 7, 2021

Conversation

kpdecker
Copy link
Collaborator

@kpdecker kpdecker commented May 7, 2021

This allows new files that are not yet tracked in git to be excluded from reporting.

kpdecker added 2 commits May 6, 2021 20:46
This allows new files that are not yet tracked in git to be excluded from reporting.
@kpdecker kpdecker force-pushed the ignore-untracked branch from e615992 to 1c8acef Compare May 7, 2021 02:06
},
)
.help().argv;
if (process.env.NODE_ENV !== 'test') {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There was a race condition in the tests with this running in the background. The addition of the additional async calls for the git logic, even though mocked, exposed this, leading to inconsistent behavior within the jest tests.

@smeijer smeijer force-pushed the ignore-untracked branch from 53c2967 to 5d85056 Compare May 7, 2021 08:05
@smeijer
Copy link
Owner

smeijer commented May 7, 2021

Nice one! Thanks!

Also, I've added you as a collaborator on the project. Please continue to make PRs as you feel the need (you can make your branches directly on the repo rather than your fork if you want).

Welcome to the team 🎉

@smeijer smeijer changed the title Add --ignore-untracked CLI arg feat: add option to ignore untracked files May 7, 2021
@smeijer smeijer merged commit 1cb6083 into smeijer:master May 7, 2021
@smeijer
Copy link
Owner

smeijer commented May 7, 2021

published as 1.11.0 🎉

@kpdecker kpdecker deleted the ignore-untracked branch July 5, 2021 15:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants