Skip to content

Conversation

@dvoituron
Copy link
Collaborator

[Fundamental] PART 1 - Editor.Config update and code fixing

Update existing editor.config file using Aspire and AspNetCore config files.

Some resources:

Content

  • Update EditorConfig
  • Add CodeAnalysis Libs
    • Microsoft.VisualStudio.Threading.Analyzers
    • StyleCop.Analyzers (not yet activate)

Fixes:

  • Add file header MIT License - Copyright (c) Microsoft Corporation. All rights reserved. (not yet applied to files)
  • Fix Whitespace, EndOfLine and File Encoding
  • IDE0005: Unnecessary using directives
  • IDE0007: 'var' preferences
  • IDE0011: Add braces
  • IDE0031: Use null propagation
  • IDE0044: Add readonly modifier
  • IDE0048: Parentheses preferences
  • IDE0160: Namespace declaration preferences
  • IDE2000: Disallow multiple blank lines
  • VSTHRD200: Use Async suffix for async methods (not Public)

⚠️ No breaking changes, no updated public methods and no updated unit tests.

@dvoituron dvoituron requested a review from vnbaaij January 26, 2024 16:22
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1389.westeurope.3.azurestaticapps.net

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1389.westeurope.3.azurestaticapps.net

@dvoituron dvoituron enabled auto-merge (squash) January 27, 2024 09:32
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1389.westeurope.3.azurestaticapps.net

@dvoituron dvoituron disabled auto-merge January 27, 2024 19:51
@dvoituron dvoituron enabled auto-merge (squash) January 27, 2024 19:52
@dvoituron dvoituron disabled auto-merge January 27, 2024 20:03
@dvoituron dvoituron merged commit 1c6feb8 into dev Jan 27, 2024
@dvoituron dvoituron deleted the users/dvoituron/fundamentals/editor-config branch January 27, 2024 20:03
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.

2 participants