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

docs: add os related scripts #4153

Closed
wants to merge 1 commit into from
Closed

Conversation

franzudev
Copy link

While composing my dotfiles and searching around, I've found out this useful feature in twpayne/dotfiles - .chezmoiscripts, but I've found that it is not documented.
I hope this can help.

@bradenhilton
Copy link
Collaborator

This isn't correct. Tom controls the execution of these scripts with a .chezmoiignore file.

@franzudev
Copy link
Author

franzudev commented Dec 22, 2024

Of course I've added those lines in my dotfiles, but forgot about them 😅
If anyone thinks it can be valuable to document this behavior I can add the .chezmoiignore part, let me know.

@twpayne
Copy link
Owner

twpayne commented Dec 27, 2024

If anyone thinks it can be valuable to document this behavior I can add the .chezmoiignore part, let me know.

Yup, although it's a "neat trick" that uses .chezmoiignore rather than any special chezmoi behavior. Please do open a PR that documents this trick -- it should be added to the user guide rather than the reference manual. In the meantime, I'll close this PR.

@twpayne twpayne closed this Dec 27, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 4, 2025
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.

3 participants