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

Standardizes additional file types in .editorconfig #1266

Merged
merged 3 commits into from
Dec 9, 2023

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