Skip to content

Conversation

@christianlupus
Copy link
Contributor

This will create an aliases file that can be sourced in the .bashrc. It can be seen as a draft idea and extended as needed.

@christianlupus
Copy link
Contributor Author

@juliusknorr this PR is quite old. Would we better close this or try to get this merged? Is i to invasive? Just give me your impression, then I can prepare this accordingly.

@juliusknorr
Copy link
Owner

I'm not sure if this is too much magic and we should rather just document useful aliases, what do you think?

@christianlupus
Copy link
Contributor Author

Yeah, generally, this can be put to the docs as well. In any case, there needs to be something in the docs on how to install the alias file.

One point is that this will not auto-update over time if done in the docs (the users will not redo it regularly). When it is just linked/sourced, we could change the alias definitions in case the aliases change (e.g. conversion to function etc). Just out of maintenance perspective.

@juliusknorr
Copy link
Owner

RIght, with a way to update that is certainly nice. The code itself seems good, then we can get it merged form my side once we have a short documentation addition.

@christianlupus
Copy link
Contributor Author

So, my next tasks as rebase, write docs and ping you again. Correct? Then, I would go this route (just wanted to confirm this to avoid unnecessary work in general).

@juliusknorr
Copy link
Owner

Yes, that would be much appreciated. :) Thanks already and sorry for the delay with checking out.

@christianlupus christianlupus marked this pull request as draft March 25, 2025 18:19
Signed-off-by: Christian Wolf <[email protected]>
Signed-off-by: Christian Wolf <[email protected]>
@christianlupus christianlupus marked this pull request as ready for review March 25, 2025 19:01
@christianlupus
Copy link
Contributor Author

@juliusknorr I added some documentation. Do you think this is sufficient or do we need more? I tried to add --help as well.

Just give me a shot notice. Thanks!

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