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

feat: add aconfmgr cd command #165

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gardar
Copy link

@gardar gardar commented Jun 9, 2023

Command to quickly change to the aconfmgr config directory, useful when managing the aconfmgr config with git where you need to enter the aconfmgr config directory to commit/push your changes.

Idea came from chezmoi which I use to manage my dotfiles.

Signed-off-by: gardar <gardar@users.noreply.github.com>
@CyberShadow
Copy link
Owner

Thank you for the contribution. However, I'm sorry but I don't like this.

I think the wiki would be a good place for the cd helper though.

@gardar
Copy link
Author

gardar commented Jun 9, 2023

Ok no worries, I wish it were possible to cd from a bash script, printing the instructions was the next best thing I could think of.
Want me to update the PR to remove it from the script and just leave the docs on how to do it?

@CyberShadow
Copy link
Owner

Well, I think the wiki would be the best place to host such utilities, which are useful but perhaps not for all users.

Granted, the wiki could be more visible in the documentation. Maybe something like this:

diff --git a/README.md b/README.md
index 33703c8..964899c 100644
--- a/README.md
+++ b/README.md
@@ -31,6 +31,7 @@ Since the system configuration is described as shell scripts, it is trivially ex
     * [Inlining file content entirely](#inlining-file-content-entirely)
     * [Inlining file edits](#inlining-file-edits)
   * [Managing multiple systems](#managing-multiple-systems)
+  * [More tips and tricks](#more-tips-and-tricks)
 * [Comparisons](#comparisons)
   * [aconfmgr vs. Puppet/Ansible](#aconfmgr-vs-puppetansible)
   * [aconfmgr vs. NixOS](#aconfmgr-vs-nixos)
@@ -281,6 +282,10 @@ CopyFileTo "/etc/hosts-$HOSTNAME" "/etc/hosts"
 
 This will use `files/etc/hosts-home.example.net` for the `home.example.net` machine, and so on.
 
+### More tips and tricks
+
+You can find user-contributed additions on [the project wiki](https://github.com/CyberShadow/aconfmgr/wiki).
+
 ## Comparisons
 
 ### aconfmgr vs. Puppet/Ansible

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