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 clarification about all lowercase custom data fields #2011

Merged
merged 1 commit into from
Apr 14, 2022

Conversation

skaldarnar
Copy link
Contributor

@skaldarnar skaldarnar commented Apr 13, 2022

Hi,

I'm new to chezmoi and started to look into templates. I've spent quite some time to figure out why I could not get custom variables in the data section of the config to work until I realized that I've stumbled upon #463.

I propose to add a hint or disclaimer to the documentation here to make it easier for newcomers to spot this pitfall (and avoid it). The FAQ added in #671 contains a reference already, but I only found it by backtracking from the issue.

Maybe adding the hint (also) to User Guide > Templating would also help?

Please let me know if I need to change or update anything.

Hi, 

I'm new to chezmoi and started to look into templates. I've spent quite some time to figure out why I could not get custom variables in the `data` section of the config to work until I realized that I've stumbled upon twpayne#463.

I propose to add a hint or disclaimer to the documentation here to make it easier for newcomers to spot this pitfall (and avoid it).

Please let me know if I need to change or update anything.
@twpayne twpayne merged commit cbfe388 into twpayne:master Apr 14, 2022
@twpayne
Copy link
Owner

twpayne commented Apr 14, 2022

This is great, thank you!

Maybe adding the hint (also) to User Guide > Templating would also help?

Such a change would be welcome too :)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 23, 2024
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