-
-
Notifications
You must be signed in to change notification settings - Fork 253
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
Adding "Directory.Build.props" and ".editorconfig" #345
Comments
So, TBH the duplicates in Let's not add the sonar analyzer yet though. Would be more appropriate to add that in a separate PR as:
LMK once a PR is ready 👍 Thanks! |
@jamesmh , The pull request with number #346 is now ready. Our next step could be adding If you are happy with that, I can create another issue for |
Some of the configuration settings for each project are shared, such as
Authors
andCompany
, which are repeated in every project. Moreover, we may want to enable the nullability feature in all projects, but with the current configuration, we would have to add theNullable
setting individually to each project, which is not efficient. A solution for this issue is to create aDirectory.Build.props
file at the root of the project and include all the common configuration settings in that file.For example, the file could look like this:
In this file, I have added the shared configuration settings and enabled the Nullble feature for all projects. I have also added
SonarAnalyzer.CSharp
code analysis to all projects to maintain high code quality.Additionally, we can create a ".editorconfig" file with the default configuration suggested by Microsoft to ensure a consistent code style across the entire solution.
The text was updated successfully, but these errors were encountered: