Skip to content

do not warn, if valid [UnrealSDK]#3898

Merged
bfops merged 1 commit intoclockworklabs:masterfrom
KirstenWF:warn_only_if_not_valid
Dec 19, 2025
Merged

do not warn, if valid [UnrealSDK]#3898
bfops merged 1 commit intoclockworklabs:masterfrom
KirstenWF:warn_only_if_not_valid

Conversation

@KirstenWF
Copy link
Contributor

Description of Changes

Avoid warning if table is valid.

API and ABI breaking changes

Expected complexity level and risk

1

Testing

Accessing tables via Blueprint and no warning in output log, if table is valid.

@KirstenWF KirstenWF changed the title do not warn, if valid do not warn, if valid [UnrealSDK] Dec 17, 2025
Copy link
Contributor

@JasonAtClockwork JasonAtClockwork left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solid call on gating the logging behind invalid data. Thanks for the commit!

@bfops bfops enabled auto-merge December 18, 2025 17:47
@bfops bfops added this pull request to the merge queue Dec 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 18, 2025
@bfops bfops added this pull request to the merge queue Dec 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 18, 2025
@bfops bfops added this pull request to the merge queue Dec 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 18, 2025
@bfops bfops enabled auto-merge December 19, 2025 00:05
@bfops bfops disabled auto-merge December 19, 2025 00:07
@bfops bfops added this pull request to the merge queue Dec 19, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 19, 2025
@bfops bfops added this pull request to the merge queue Dec 19, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 19, 2025
@bfops bfops added this pull request to the merge queue Dec 19, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 19, 2025
@bfops bfops added this pull request to the merge queue Dec 19, 2025
Merged via the queue into clockworklabs:master with commit df035c4 Dec 19, 2025
65 of 68 checks passed
@KirstenWF KirstenWF deleted the warn_only_if_not_valid branch January 27, 2026 13:31
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.

3 participants