Skip to content

DEVELOPMENT_STANDARDS.md Proposed - #2

Open
antoineatstariongroup wants to merge 2 commits into
developmentfrom
feat/standard-guidelines
Open

DEVELOPMENT_STANDARDS.md Proposed#2
antoineatstariongroup wants to merge 2 commits into
developmentfrom
feat/standard-guidelines

Conversation

@antoineatstariongroup

Copy link
Copy Markdown

The PR provides a first version of the Development standards that CLAUDE Code shall follow during C# development.
I would ask any that review this to provide comments about missing parts, potential improvement or if some pieces are overkill or need to be changed.

Comment thread DEVELOPMENT_STANDARDS.md

---

## 1. C# code style

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think part of this can be configured through code analysis or linting, see https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis. This will avoid setting conflicting standards between claude, code analysis, sonar and the IDE settings.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Agreed, but it would be nice to have also a common code style for Starion then. If one exist, we can setup Claude to follow that one

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think that is blocking for this issue. Currently I use the sonar rules as is. I have claude checking sonar. That is very strict. I do not care a lot about standards besides that. But I don't think it is good idea to instruct claude to use a standard that conflicts with project CI settings liket the sonar check or other common code analysis rules.

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