Skip to content

Fix instruction.md: prose clarity, typo, and correct GlobalExceptionHandler reference#66

Closed
Copilot wants to merge 2 commits intogithub-copilot-instruction-filefrom
copilot/sub-pr-65
Closed

Fix instruction.md: prose clarity, typo, and correct GlobalExceptionHandler reference#66
Copilot wants to merge 2 commits intogithub-copilot-instruction-filefrom
copilot/sub-pr-65

Conversation

Copy link

Copilot AI commented Mar 5, 2026

Addresses review feedback on the Copilot instruction file — typo, run-on prose, and a stale component reference that would mislead contributors.

Changes

  • DataEngine description (line 7): Replaced run-on paragraph (including "from from" duplicate) with three concise bullet points describing the template-load → plugin-resolve → fill-and-return flow.
  • Registry/Repository sentence (line 22): Rephrased to remove awkward capitalization and passive wording; now reads as a single clear sentence.
  • Error Handling section (lines 97–101): Replaced ErrorController with GlobalExceptionHandler (IExceptionHandler) to match the actual implementation — both in the usage guidance and the base-exception-mapping note.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…nHandler updates

Co-authored-by: mm-kgi <229314169+mm-kgi@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on instruction file for DataEngine project architecture Fix instruction.md: prose clarity, typo, and correct GlobalExceptionHandler reference Mar 5, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 5, 2026

@mm-kgi mm-kgi closed this Mar 5, 2026
@mm-asha mm-asha deleted the copilot/sub-pr-65 branch March 19, 2026 12: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