Conversation
…rconfig docker :latest) (#401) Converges Utilities onto the fleet lint architecture: editorconfig-checker via docker :latest (its action is install-only), VS Code Lint group added, husky language-only with the if guard, cspell scoped to README+HISTORY, AGENTS.md lint guidance synced.
Fleet lint conventions: EditorConfig rename (step + VS Code task) and process-type VS Code Lint tasks. Canonical: ProjectTemplate #280.
Contributor
There was a problem hiding this comment.
Pull request overview
Promotes the accumulated conformance and tooling updates from develop to main, focusing on test infrastructure (xUnit v3), lint surfaces (CI + VS Code tasks), and local hook behavior, without changing library source under Utilities/**.
Changes:
- Migrate the test project to xUnit v3 (update package references, simplify test project packaging metadata).
- Expand local lint “surfaces” via VS Code tasks (Docker
:latestwith--pull=always) and align CI wording/step naming for EditorConfig checks. - Refine local pre-commit behavior to run Husky.Net only when
dotnetis available; remove an obsolete CA2007 suppression in test.editorconfig.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
UtilitiesTests/UtilitiesTests.csproj |
Switch test dependencies to xunit.v3 and simplify test project properties. |
UtilitiesTests/.editorconfig |
Remove CA2007 suppression from test analyzer overrides. |
Directory.Packages.props |
Replace xunit v2 version pin with xunit.v3 version pin. |
AGENTS.md |
Update documented lint tooling conventions (CI vs local, :latest usage). |
.vscode/tasks.json |
Add a “Lint” task group running doc/workflow linters via Docker with --pull=always. |
.husky/pre-commit |
Guard Husky.Net invocation behind command -v dotnet and clarify intent in comments. |
.github/workflows/validate-task.yml |
Rename EditorConfig step and run editorconfig-checker via Docker :latest. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Forward promotion of the accumulated conformance work (#401/#402/#403). No release: this is a human merge, so the publish plan gates to publish=false. No library source (
Utilities/**) changed.