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

Automate C# formatting using dotnet-format #19

Open
imback82 opened this issue Feb 18, 2019 · 8 comments
Open

Automate C# formatting using dotnet-format #19

imback82 opened this issue Feb 18, 2019 · 8 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@imback82
Copy link
Contributor

Explore adopting https://github.com/dotnet/format to automate the process of formatting source code similar to scalafmt (#153).

@rapoth rapoth transferred this issue from another repository Apr 24, 2019
@rapoth rapoth added the good first issue Good for newcomers label Apr 24, 2019
@nmnick18
Copy link

I'd like to take up this issue

@imback82
Copy link
Contributor Author

Cool, that will be great. Please let me know if we can retain most of the existing formatting or not. Thanks!

@nmnick18
Copy link

nmnick18 commented Jan 24, 2021

I am gonna need more context, but is this mainly looking to use some of the 'format' code to automatically format the scala code?

Bare with me here lol, this is my first open source contribution (yay!)

@postmeback
Copy link

Any update on this ?

@ghostp13409
Copy link

Hey, is this issue still available to work on? I would like to give it a try.

@wudanzy
Copy link
Collaborator

wudanzy commented Dec 23, 2024

Seem like no progress within 3 years, please feel free to have a try!

@ghostp13409
Copy link

ghostp13409 commented Dec 24, 2024

Thanks. I'll get started with installing dotnet/Format into the project. However, I do have a few doubts.

  1. the issue given for reference leads to some feature request Support Trigger in DataStreamWriter. I don't understand how am i supposed to see scalafmt things in there.
  2. I found a C# Coding Style guideline. Do you want me to implement those rules in .editorconfig file? because those things already seems to be in place.
  3. Just to be sure, you want me to only work on src/csharp/ and not anything else, correct?
  4. I couldn't find any guideline for how should i make the contribution. should I just work on the main, create a new branch or create my own fork for the issue?

@wudanzy
Copy link
Collaborator

wudanzy commented Dec 30, 2024

For 1, I don't have much context there, I would guess that code was formatted using that formatter.
For 2, I think you understanding is right. We can use some rule so that any future code can have the same style, but you may need some exploration, I am new to C#. Please check how that's done in other C# repos and check if this item was obsoleted. It is a request 6 years ago.
For 3, C# changes are only in that directory. You can contribute to other directories if you have other ideas.
For 4, usually you need to fork on to your own repo and using a dev branch, after you are done,you create a PR to request merging to the main branch of dotnet spark.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

6 participants