Skip to content

Shorten error codes from 4-digit to 3-digit padding#25

Merged
304NotModified merged 3 commits into
mainfrom
copilot/update-error-codes-to-three-digits
Mar 7, 2026
Merged

Shorten error codes from 4-digit to 3-digit padding#25
304NotModified merged 3 commits into
mainfrom
copilot/update-error-codes-to-three-digits

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 7, 2026

  • Update ValidationErrorCodeExtensions.cs: change D4 to D3
  • Update SlnxValidatorTests.cs: update expected codes (SLNX0001 → SLNX001, etc.)
  • Update README.md: update error codes table and examples
  • Update examples/invalid-not-xml.slnx: update error code reference (SLNX0010 → SLNX010)
  • All 17 tests pass
  • CodeQL: no alerts

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: 304NotModified <5808377+304NotModified@users.noreply.github.com>
Copilot AI changed the title [WIP] Change error codes to three digits and update tests and docs Shorten error codes from 4-digit to 3-digit padding Mar 7, 2026
@304NotModified
Copy link
Copy Markdown
Owner

@copilot also check the examples files

Co-authored-by: 304NotModified <5808377+304NotModified@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 7, 2026

@copilot also check the examples files

Fixed in cb532cfexamples/invalid-not-xml.slnx had SLNX0010 which is now updated to SLNX010. The other example files didn't contain any error code references.

@304NotModified 304NotModified marked this pull request as ready for review March 7, 2026 23:03
@304NotModified 304NotModified merged commit 9cd0c4c into main Mar 7, 2026
6 checks passed
@304NotModified 304NotModified deleted the copilot/update-error-codes-to-three-digits branch March 7, 2026 23:05
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