peteroupc / CBOR

A C# implementation of Concise Binary Object Representation (RFC 8949).
The Unlicense
206 stars 29 forks source link

Code style - reformat or add configuration file? #62

Closed charlesroddie closed 1 year ago

charlesroddie commented 2 years ago

Hi @peteroupc

I'd like to work on PRs for this repo. But when I load the project and make changes, it's very hard to format since the IDE auto-formatting doesn't match the code.

Solutions:

  1. Format existing code according to default C# conventions (Ctrl-K Ctrl-D on all files).
  2. Add a configuration file that matches existing formatting.

Do you have a preference for the existing formatting and would you accept a PR to do either 1 or 2? I prefer 1 but could try to figure out 2.

Is it OK to include IDE code suggestions too in a cleanup PR?