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

Add note on file types #97

Merged
merged 2 commits into from Jul 25, 2022
Merged

Add note on file types #97

merged 2 commits into from Jul 25, 2022

Conversation

vsoch
Copy link
Collaborator

@vsoch vsoch commented Jul 25, 2022

No description provided.

If you need to specify a file_types pattern to include hidden files, you'll need a `.*` pattern that is provided in the context of a comma separated list, for example:

```yaml
file_types: '.*,'
Copy link
Member

Choose a reason for hiding this comment

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

what happens here if I only want to check the hidden files? (not sure how frequent is this scenario)
based on this, I will need to exclude the other non-hidden types right?
To do that, the only option I see is exclude_files but that requires full path right? or can I use a pattern?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think if you add this file types filter it would exclude all the other patterns - in the same way doing *.md only includes markdown. But I haven’t tried it recently!

Choose a reason for hiding this comment

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

Yes, I use this on a dotfiles repo (where it's all "hidden" files) and this is the pattern I use, after guidance in #76.

Copy link
Member

Choose a reason for hiding this comment

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

Then that's perfect 👍

@vsoch vsoch merged commit 108d4b9 into master Jul 25, 2022
@vsoch vsoch deleted the add/file-types-note branch July 25, 2022 23:47
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.

None yet

3 participants