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

Doc: Update SSH Agent section #6927

Merged
merged 1 commit into from
Oct 2, 2021
Merged

Doc: Update SSH Agent section #6927

merged 1 commit into from
Oct 2, 2021

Conversation

Okeanos
Copy link
Contributor

@Okeanos Okeanos commented Sep 19, 2021

This might be a basis to document the Git-Bash ssh-socket workaround mentioned by me in #4681 as well – or would that go into the FAQ?

Type of change

  • ✅ Documentation (non-code change)

As a sidenote … would anyone of you be interested in putting an editorconfig into the repository? While resetting local editor settings to match individual repository preferences (crlf vs. lf, tabs vs spaces, …) is certainly possible, I personally hate dealing with that.

* Update naming of `macOS` to use the official case instead of `MacOS`
* Add an in depth explanation for using Windows OpenSSH (see keepassxreboot/keepassxc-org#96 for details)
* Remove some trailing whitespace
@droidmonkey
Copy link
Member

droidmonkey commented Sep 19, 2021

The git bash thing is good for an FAQ or you could just link to your issue comment. You can add the desired editor config to this pr.

@Okeanos
Copy link
Contributor Author

Okeanos commented Sep 19, 2021

I think I will open a separate PR for the editorconfig addition – I would like to test and verify these changes independently from any documentation contents and allow you as official maintainers to do so as well.

Additionally, I will update my previous FAQ pull request to include the Git Bash workaround.

@droidmonkey
Copy link
Member

Interesting enough visual studio code does not honor editorconfig without a plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants