-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add french doc #357
Add french doc #357
Conversation
@clemsau keybindings are generated via |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marking as changes requested for my own bookkeeping
As requested, I:
|
@clemsau sweet :) Looks like you'll need to rebase onto master as there were some changes since you first raised this PR |
The installation method used up until now would always leave one copy of the lazydocker binary in the extraction directory. This copy is not actually needed after installation, so it should be removed. TheAxelander also pointed this out in his comment at #270 (comment). This commit changes that behaviour and makes the install script cleanup after itself.
Add french translation for the keybindings doc.
In the
CONTRIBUTING.md
it is written that one should first discuss with the author before contributing, but considering how basic adding a translation to docs is, I assumed I could directly go for a PR.Please decline the PR if the changes are unwanted.
Regards.