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: reference templates in top-level passwd mgmt #4162

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jondkinney
Copy link

It's important for people to understand that the password manager feature is an extension/implementation of the existing templating capabilities of Chezmoi.

It's important for people to understand that the password manager
feature is an extension/implementation of the existing templating
capabilities of Chezmoi.
@jondkinney jondkinney force-pushed the password-manager-doc-improvements branch from e3023d5 to 7b8fce8 Compare December 27, 2024 19:57
Co-authored-by: Austin Ziegler <austin@zieglers.ca>
@jondkinney
Copy link
Author

@halostatue I'm not sure what you prefer for etiquette when accepting changes from a PR review/suggestion. Would you like to remain co-author for those updates? Fine with me if so, but I can also rebase it to a single commit including your suggestions while retaining the conventional commit style, if you prefer. LMK and I'll get it tweaked. Thanks!

@halostatue
Copy link
Collaborator

@halostatue I'm not sure what you prefer for etiquette when accepting changes from a PR review/suggestion. Would you like to remain co-author for those updates? Fine with me if so, but I can also rebase it to a single commit including your suggestions while retaining the conventional commit style, if you prefer. LMK and I'll get it tweaked. Thanks!

I don't need to be credited for this; just squash it into your original commit, because we prefer to have one commit per pull request.

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.

2 participants