Skip to content

Use consistent error format in delta.#580

Merged
Thomas-Boyle merged 13 commits intomasterfrom
VED-000-consistent-error-format
Apr 22, 2026
Merged

Use consistent error format in delta.#580
Thomas-Boyle merged 13 commits intomasterfrom
VED-000-consistent-error-format

Conversation

@mfjarvis
Copy link
Copy Markdown
Contributor

@mfjarvis mfjarvis commented Jun 12, 2025 •
edited by Thomas-Boyle
Loading

  • Standardised delta conversion errors to one structured format with code, field, value and message.
 Simplified converter and extractor error handling while keeping handler partial success behaviour unchanged.
  • Strengthened delta tests to verify structured diagnostics end to end, including handler propagation and warning logs.
 Removed the last legacy shaped mocked error payload; the full delta-backend suite passes with 268 tests.

@mfjarvis mfjarvis force-pushed the VED-000-consistent-error-format branch from 1c9664d to 47e1f40 Compare June 12, 2025 10:39
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 12, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Akol125
Akol125 previously approved these changes Jun 19, 2025
View reviewed changes
@dlzhry2nhs
Copy link
Copy Markdown
Contributor

dlzhry2nhs commented Oct 7, 2025

PR still needed? If not please close.

@mfjarvis mfjarvis changed the title VED-XXX: Use consistent error format in delta. Use consistent error format in delta. Oct 7, 2025
@mfjarvis mfjarvis force-pushed the VED-000-consistent-error-format branch from d20a484 to 2bcf49e Compare October 7, 2025 14:13
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Oct 7, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines Bot commented Apr 15, 2026

Azure Pipelines:
Successfully started running 1 pipeline(s).

@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 15, 2026 12:56 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 15, 2026 12:56 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 15, 2026 12:57 — with GitHub Actions Inactive
@azure-pipelines
Copy link
Copy Markdown

azure-pipelines Bot commented Apr 15, 2026

Azure Pipelines:
Successfully started running 1 pipeline(s).

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines Bot commented Apr 15, 2026

Azure Pipelines:
Successfully started running 1 pipeline(s).

@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 16, 2026 10:50 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 16, 2026 10:52 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 16, 2026 10:53 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 22, 2026 08:48 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 22, 2026 08:51 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 22, 2026 08:51 — with GitHub Actions Inactive
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 22, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Thomas-Boyle Thomas-Boyle Thomas-Boyle left review comments

@avshetty1980 avshetty1980 avshetty1980 approved these changes

@Akol125 Akol125 Akol125 approved these changes

Assignees

No one assigned

Labels

bugfix This PR contains a fix for existing functionality. python Pull requests that update python code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mfjarvis @dlzhry2nhs @avshetty1980 @Thomas-Boyle @Akol125