Skip to content

Conversation

jacobslusser
Copy link
Contributor

This is my first PR for this project and I didn't find any contributing guidelines, so please let me know if there is anything else I should do for submitting PRs.

  • Fixes the path to TestResults/ in the .gitignore. When the .sln file was moved 2 months ago, this started putting TestResults/ folder in the root instead of src/.
  • Adds formatting rules (2 space tabs) for .xml, .csproj, and .props files to .editorconfig. I also formatted those files according to these rules.
  • Adds visible=false to the stylecop.json reference in Directory.Build.props. Previously the stylecop.json file would show in every project, but now will only show in the Solution Items.

These are just simple things I found as I started getting acquainted with the code and running tests. I have bigger and better submissions planned but wanted to test the waters a little with the group and understand the process.

Copy link
Collaborator

@WojciechNagorski WojciechNagorski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing contribution! Thanks.
Welcome on board and we look forward to more!

@WojciechNagorski WojciechNagorski merged commit 4c4883e into sshnet:develop Dec 9, 2023
@Rob-Hague
Copy link
Collaborator

Thanks @jacobslusser. Please do let it be known what your plans are - the library is gaining a bit of momentum at the moment, and we probably ought to sync up a bit or start assigning ourselves to issues etc.

@WojciechNagorski WojciechNagorski added this to the 2023.0.1 milestone Dec 20, 2023
@WojciechNagorski
Copy link
Collaborator

The 2023.0.1 version has been released to Nuget: https://www.nuget.org/packages/SSH.NET/2023.0.1

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.

3 participants