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

Add .editorconfig for csharp directory #9675

Merged
merged 2 commits into from
Mar 26, 2022

Conversation

Jensaarai
Copy link
Contributor

Nicer experience for e.g. VS Code

Split from #9666

Nicer experience for e.g. VS Code

# New line preferences
# See https://github.com/protocolbuffers/protobuf/issues/9526
#end_of_line = lf

Choose a reason for hiding this comment

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

Shouldn't this be uncommented?

Copy link
Contributor

@jskeet jskeet Mar 25, 2022

Choose a reason for hiding this comment

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

Not yet - we may well do that as part of fixing #9526, but this PR just adds the editor config without trying to address that.

@jskeet
Copy link
Contributor

jskeet commented Mar 25, 2022

Please add the file to Makefile.am (along with the other C# files in csharp_EXTRA_DIST) - that will fix the Kokoro errors.

@Jensaarai
Copy link
Contributor Author

Done...for my own curiosity, what is that file for? I'm guessing it's for packaging up the e.g. protobuf-csharp-3.20.0.zip zip/tar files on the releases, for users who want to consume as source?

@jskeet
Copy link
Contributor

jskeet commented Mar 26, 2022

@Jensaarai: That's my understanding, yes. It was already in place when I started contributing to the project, and I haven't gone into great depths to check it :)

Copy link
Contributor

@jskeet jskeet left a comment

Choose a reason for hiding this comment

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

Merging this now - it'll help with #9526.

@jskeet jskeet merged commit b5a35bc into protocolbuffers:master Mar 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants