Skip to content

test: update conformance snapshots for merge conflict diagnostics#15462

Merged
sapphi-red merged 2 commits intoc/11-04-feat_parser_improve_diagnostic_messages_for_merge_conflictsfrom
copilot/sub-pr-15443
Nov 8, 2025
Merged

test: update conformance snapshots for merge conflict diagnostics#15462
sapphi-red merged 2 commits intoc/11-04-feat_parser_improve_diagnostic_messages_for_merge_conflictsfrom
copilot/sub-pr-15443

Conversation

Copy link
Contributor

Copilot AI commented Nov 8, 2025

Updates test snapshots to reflect the improved merge conflict diagnostic messages implemented in previous commits.

Changes

  • Updated parser_misc.snap and parser_typescript.snap with capitalized error titles and properly formatted help text
  • Snapshots now match the current diagnostic output: "Encountered diff marker" with sentence-case help messages

The snapshots were updated by running just conformance after the diagnostic message improvements were merged.


💡 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.

Copilot AI self-assigned this Nov 8, 2025
Co-authored-by: sapphi-red <49056869+sapphi-red@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve diagnostic messages for merge conflicts in parser test: update conformance snapshots for merge conflict diagnostics Nov 8, 2025
Copilot AI requested a review from sapphi-red November 8, 2025 06:21
@sapphi-red sapphi-red marked this pull request as ready for review November 8, 2025 06:21
@sapphi-red sapphi-red merged commit da6cf0f into c/11-04-feat_parser_improve_diagnostic_messages_for_merge_conflicts Nov 8, 2025
22 of 36 checks passed
@sapphi-red sapphi-red deleted the copilot/sub-pr-15443 branch November 8, 2025 06:29
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