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

Better way to use alias bat instead of batcat symlink for ubuntu #3040

Open
anand-dragon opened this issue Jul 24, 2024 · 2 comments
Open

Better way to use alias bat instead of batcat symlink for ubuntu #3040

anand-dragon opened this issue Jul 24, 2024 · 2 comments

Comments

@anand-dragon
Copy link

Hi,
I saw your README page and the section Installation where you have for Ubuntu an idea of using symlink to ~/.local/bin rather than creating symlinks isn't it way better to just use alias. For example on my system I have Zsh so I just did alias bat="batcat" and that was it. IMO it seems a better and fuzz-free idea.

I read your contributing documentation but was't sure how the README updates would work out so created a new issue.

@anand-dragon
Copy link
Author

Since this is my first open source contribution, some guidelines to push my changes to remote would be appreciated.

@keith-hall
Copy link
Collaborator

keith-hall commented Jul 27, 2024

Hi,

Readme updates work the same way as any other changes 🙂 just there is no need to test locally as tests shouldn't be affected 😉 and as mentioned in the contributing guide, no need to add a changelog. (You likely will see that the changelog CI step failed but that is expected and fine.)
You would fork the repository, clone it locally, create a branch in your fork, make your changes and commit them, and push them to your remote. Then in this repo on GitHub's website, where we are now, create a Pull Request, click compare across forks, with sharkdp/bat as the base and choose your fork's branch as the head.

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

No branches or pull requests

2 participants