Skip to content

fix: exit processing xml files that fail XSD - #171

Merged
stevenhsd merged 2 commits into
release_v010from
refactor/gr-ndsp-669_fix_bugs_and_improve_feedback_around_ft_errors
Oct 5, 2026
Merged

stevenhsd merged 2 commits into
release_v010from
refactor/gr-ndsp-669_fix_bugs_and_improve_feedback_around_ft_errors

Conversation

@georgeRobertson

@georgeRobertson georgeRobertson commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

TLDR of changes

  • fix: exit processing xml files that fail XSD

What kind of changes does this PR introduce?

Tick all that apply

  • fix: A bug fix. Correlates with PATCH in SemVer
  • feat: A new feature. Correlates with MINOR in SemVer
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing or correcting existing tests
  • build: Changes that affect the build system or external dependencies (example scopes: pip, docker, npm)
  • ci: Changes to CI configuration files and scripts (example scopes: GitLabCI)

Please check if the PR fulfills these requirements

  • I have read and followed the Contributing guidance
  • Docs have been added / updated
  • Tests and Linting in the CI are passing
  • Changes have been reviewed and approved by a Project Maintainer

number of feedback messages to reduce and removal of specific error code+messages for certain readers to become generic ft_error_code/message
@georgeRobertson
georgeRobertson marked this pull request as ready for review October 5, 2026 09:30
@sonarqubecloud

sonarqubecloud Bot commented Oct 5, 2026

Copy link
Copy Markdown

@stevenhsd
stevenhsd merged commit 9415d2a into release_v010 Oct 5, 2026
4 checks passed
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