Move usings to top of file and using file-scoped namespaces #40
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a mechanical change to:
usings
to the top of the fileThese two refactorings are done together because they touch every file and would otherwise be a source of merge conflicts.
In order to enforce these changes going forward, I made the following changes to
.editorconfig
:dotnet_separate_import_directive_groups = false
to not require the blank space between "1st party" and "3rd party" usingscsharp_using_directive_placement = outside_namespace:warning
to a warningcsharp_style_namespace_declarations = file_scoped:warning
as a warning