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.

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

Feat: Adjust find_rej_files() with argument for finding hidden .rej files #4

Merged
merged 15 commits into from
Jul 23, 2024

Conversation

StephanLoor
Copy link
Contributor

We currently aren't checking for hidden .rej files (i.e. filenames starting with .). This functionality is desirable, as the config of e.g. pre-commit hooks and cruft are maintained in hidden files by default. This PR adds this functionality.

@StephanLoor StephanLoor changed the title Adapted find_rej_files to also return hidden .rej files. Adapted find_rej_files() to also return hidden .rej files. Jul 17, 2024
Copy link
Owner

@MarkusSagen MarkusSagen left a comment

Choose a reason for hiding this comment

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

Thank you for proposing this change!
It's a good suggestion, but could we add a flag to typer to optionally extend with the hidden files? The flag can be true by default to show hidden files, ex. --show-hidden?

@MarkusSagen MarkusSagen changed the title Adapted find_rej_files() to also return hidden .rej files. Feat: Adjust find_rej_files() with argument for finding hidden .rej files Jul 18, 2024
@StephanLoor
Copy link
Contributor Author

StephanLoor commented Jul 18, 2024

@MarkusSagen good idea - done!

@MarkusSagen
Copy link
Owner

Great change! Merging

@MarkusSagen MarkusSagen merged commit add0d6d into MarkusSagen:main Jul 23, 2024
2 checks passed
@timvink
Copy link

timvink commented Aug 9, 2024

@MarkusSagen Would you mind making a new release to pypi with these changes also? Latest version on pypi is 0.0.3 (https://pypi.org/project/rejx/#history)

@timvink
Copy link

timvink commented Oct 4, 2024

@MarkusSagen sorry to ping you again.. would be great if you can make a pypi release

@MarkusSagen
Copy link
Owner

Sorry! Been a bit busy and had vacation
Aiming to make a new release this weekend

@MarkusSagen
Copy link
Owner

@timvink @StephanLoor Included now in the 0.2.0 release

@timvink
Copy link

timvink commented Oct 12, 2024

Thanks!

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.

3 participants