Skip to content

fix: ensure empty string rather than None or entity name is present in report for Data Item Submission Name - #201

Merged
stevenhsd merged 1 commit into
release_v0101from
fix/gr-ndsp-736-ensure_data_item_sub_name_empty_for_check_entity
Nov 4, 2026
Merged

stevenhsd merged 1 commit into
release_v0101from
fix/gr-ndsp-736-ensure_data_item_sub_name_empty_for_check_entity

Conversation

@georgeRobertson

@georgeRobertson georgeRobertson commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

TLDR of changes

ensure empty string rather than None or entity name is present in report for Data Item Submission Name and add Empty Entity to category as well for consistency with other entity check validations

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

…n report for Data Item Submission Name

add Empty Entity to category as well for consistency with other entity check validations
@sonarqubecloud

sonarqubecloud Bot commented Nov 3, 2026

Copy link
Copy Markdown

@georgeRobertson
georgeRobertson marked this pull request as ready for review November 3, 2026 16:17
@stevenhsd
stevenhsd merged commit 7799897 into release_v0101 Nov 4, 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