Refresh 6 stale instruction files (#2133)#2139
Open
KomalSrinivasan wants to merge 1 commit into
Open
Conversation
Contributor
🔒 PR Risk Scan ResultsScanned 7 changed file(s).
✅ No matching risk patterns were detected in changed files.
|
Targeted refresh: - blazor: C# 13 to C# 14. Drop the Visual Studio Enterprise mandate so contributors on VS Code or Rider aren't blocked by a paid SKU. Swap VS-only profiling for dotnet-trace and dotnet-counters. - copilot-thought-logging: narrow applyTo from '**' to '**/Copilot-Processing.md'. Replace 9 Windows backslash paths with POSIX './Copilot-Processing.md' so the workflow works on macOS and Linux. - genaiscript: drop the "avoid exception handlers or error checking" line. Replace it with: handle errors at I/O and external API boundaries, let unexpected exceptions surface. - memory-bank: add the required 'description' frontmatter field (was a validation failure). Narrow applyTo from '**' to 'memory-bank/**'. Add an opt-in note so contributors know auxiliary files land in the workspace root. Minor modernization: - azure-functions-typescript: Node.js v20 to v22 LTS. - localization: relative '../../issues' disclaimer link to absolute https://github.com/github/awesome-copilot/issues so it resolves regardless of the localized doc's path. docs/README.instructions.md regenerated by 'npm run build' to pick up the new memory-bank description.
KomalSrinivasan
force-pushed
the
fix/issue-2133-stale-resource-refresh
branch
from
a0bf675 to
45edc81
Compare
KomalSrinivasan
changed the title
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
1 participant
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.
Picks up the 6 instruction files the Resource Staleness Report deep-reviewed and flagged with concrete problems. 4 "Targeted refresh" + 2 "Minor modernization". The remaining 716 stale files are bulk batch-commit noise per the report itself, so they stay out.
Changes
Targeted refresh
instructions/blazor.instructions.mddotnet-trace/dotnet-counters, which work cross-platform.instructions/copilot-thought-logging.instructions.mdapplyTofrom**(every file in the repo) to**/Copilot-Processing.md.\Copilot-Processing.mdpaths with POSIX./Copilot-Processing.mdso the workflow runs on macOS and Linux.instructions/genaiscript.instructions.mdinstructions/memory-bank.instructions.mddescriptionfrontmatter (it was missing, which is a validation failure perAGENTS.md).applyTofrom**tomemory-bank/**so it only fires in repos that actually use the pattern.Minor modernization
instructions/azure-functions-typescript.instructions.mdinstructions/localization.instructions.md../../issueslink in the disclaimer to absolutehttps://github.com/github/awesome-copilot/issuesso it resolves regardless of where the localized doc sits.Auto-generated
docs/README.instructions.mdgot regenerated bynpm run buildto pick up the newmemory-bankdescription. CI requires this stay in sync.Verification
npm install && npm run buildexits 0. Generatedmarketplace.jsonwith 96 plugins.grep -n '\\Copilot-Processing' instructions/copilot-thought-logging.instructions.mdreturns 0 matches.Out of scope
bicep-code-best-practices,cmake-vcpkg,nextjs-tailwind,generate-modern-terraform-code-for-azure. The report explicitly says "no urgent change needed" on these.Closes #2133 partial.