Skip to content

fix: fix failures not being properly audited in foundry pipeline#94

Merged
stevenhsd merged 2 commits intomainfrom
hotfix/gr-ndit-1236_issue_with_failed_processing_status_not_being_correctly_recorded
Apr 17, 2026
Merged

fix: fix failures not being properly audited in foundry pipeline#94
stevenhsd merged 2 commits intomainfrom
hotfix/gr-ndit-1236_issue_with_failed_processing_status_not_being_correctly_recorded

Conversation

@georgeRobertson
Copy link
Copy Markdown
Contributor

TLDR of changes

Fix issue where submission status was not being properly recorded in the foundry pipeline causing downstream process to incorrectly record the submission as successful.

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

@sonarqubecloud
Copy link
Copy Markdown

@stevenhsd stevenhsd merged commit 61c0523 into main Apr 17, 2026
4 checks passed
@georgeRobertson georgeRobertson deleted the hotfix/gr-ndit-1236_issue_with_failed_processing_status_not_being_correctly_recorded branch April 29, 2026 09:42
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