Skip to content

Address review feedback: fix stderr handling, validation, and editorconfig #14403

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.

Sign up for GitHub

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

Draft
Copilot wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Address review feedback: fix stderr handling, validation, and editorconfig #14403

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
8a78cf6
Add dependency-update skill for external NuGet package updates
Feb 8, 2026
0be42c8
Rewrite migrate-package script as single-file C# app
Feb 8, 2026
c4dc961
Update Hex1b packages to 0.75.0 and add Hex1b.Tool
Feb 9, 2026
2d73608
Update .github/skills/dependency-update/MigratePackage.cs
mitchdenny Feb 9, 2026
5a45c9a
Initial plan
Copilot Feb 9, 2026
eb8e86d
Address PR review feedback: fix editorconfig, add validation, handle …
Copilot Feb 9, 2026
b68c018
Improve async handling: await tasks directly instead of using .Result
Copilot Feb 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .config/dotnet-tools.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@
"commands": [
"pwsh"
]
},
"hex1b.tool": {
"version": "0.75.0",
"commands": [
"hex1b"
]
}
}
}
10 changes: 10 additions & 0 deletions .github/skills/dependency-update/.editorconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# Override repo-level analyzers for standalone tool scripts
[*.cs]
# Disable file header requirement
dotnet_diagnostic.IDE0073.severity = none
# Disable unused using warning (script may need conditional usings)
dotnet_diagnostic.IDE0005.severity = suggestion
# Disable ConfigureAwait requirement (not needed in console apps)
dotnet_diagnostic.CA2007.severity = none
# Disable locale-sensitive parsing warning
dotnet_diagnostic.CA1305.severity = none
8 changes: 8 additions & 0 deletions .github/skills/dependency-update/Directory.Packages.props
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
<Project>
<!-- Override the repo-level Central Package Management for single-file C# scripts in this directory.
The #:package directives in .cs files generate a csproj with inline PackageReference versions,
which conflicts with CPM. -->
<PropertyGroup>
<ManagePackageVersionsCentrally>false</ManagePackageVersionsCentrally>
</PropertyGroup>
</Project>
Loading