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

ignore based on package dirent name prior to dot- adjustment #116

Merged
merged 3 commits into from
Sep 8, 2024

Conversation

aspiers
Copy link
Owner

@aspiers aspiers commented Sep 8, 2024

If --dotfiles was enabled, files in the package such as dot-gitignore would be translated to .gitignore and then ignored by the default ignore list. However any file named dot-* in a package is obviously intended to be stowed as a dot file, so should not be ignored.

To fix this, ignore based on the name in the package, not the potentially translated name used for stowing.

This was first reported in #75 (comment).

@aspiers aspiers mentioned this pull request Sep 8, 2024
@aspiers aspiers force-pushed the fix-dot-ignore branch 3 times, most recently from 2dbf039 to c3ae95f Compare September 8, 2024 15:28
If `--dotfiles` was enabled, files in the package such as
`dot-gitignore` would be translated to `.gitignore` and then ignored
by the default ignore list.  However any file named `dot-*` in a
package is obviously intended to be stowed as a dot file, so should
not be ignored.

To fix this, ignore based on the name in the package, not the
potentially translated name used for stowing.
@aspiers aspiers self-assigned this Sep 8, 2024
@aspiers aspiers added the bug label Sep 8, 2024
@aspiers aspiers merged commit 94a895e into master Sep 8, 2024
6 checks passed
@aspiers aspiers deleted the fix-dot-ignore branch September 8, 2024 18:05
@aspiers
Copy link
Owner Author

aspiers commented Sep 8, 2024

Now released in 2.4.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant