Skip to content

[Test]#673

Open
lopezt-unity wants to merge 1 commit into
masterfrom
test/dummy
Open

[Test]#673
lopezt-unity wants to merge 1 commit into
masterfrom
test/dummy

Conversation

@lopezt-unity
Copy link
Copy Markdown
Collaborator

DO NOT FORGET TO INCLUDE A CHANGELOG ENTRY

Purpose of this PR

[Brief summary of the changes in this PR.]

Links

Jira:

Comments to Reviewers

[List known issues, planned work, provide any extra context for your code.]

Copy link
Copy Markdown
Contributor

@u-pr u-pr Bot left a comment

Choose a reason for hiding this comment

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

Perfect

Thank you for the contribution! The changes look very solid. We identified only a single minor, low-importance item regarding a potential temporary placeholder entry in the changelog.

🤖 Helpful? 👍/👎

Comment thread CHANGELOG.md

## Unreleased

[Test]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

low

It looks like [Test] might be a temporary placeholder entry in the changelog. Have you considered replacing it with a description of the actual changes, or removing it if it was added for testing purposes?

🤖 Helpful? 👍/👎 by guardian

@codecov-github-com
Copy link
Copy Markdown

codecov-github-com Bot commented May 28, 2026

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff             @@
##           master     #673      +/-   ##
==========================================
+ Coverage   36.05%   37.58%   +1.52%     
==========================================
  Files         277      278       +1     
  Lines       34909    38779    +3870     
==========================================
+ Hits        12588    14575    +1987     
- Misses      22321    24204    +1883     
Flag Coverage Δ
probuilder_MacOS_6000.3 35.14% <ø> (+0.56%) ⬆️
probuilder_MacOS_6000.4 35.13% <ø> (+0.56%) ⬆️
probuilder_MacOS_6000.5 35.14% <ø> (+0.56%) ⬆️
probuilder_MacOS_6000.6 35.14% <ø> (?)
probuilder_Windows_6000.0 35.06% <ø> (-0.19%) ⬇️
probuilder_Windows_6000.3 35.06% <ø> (-0.19%) ⬇️
probuilder_Windows_6000.4 35.05% <ø> (-0.19%) ⬇️
probuilder_Windows_6000.5 35.06% <ø> (-0.19%) ⬇️
probuilder_Windows_6000.6 35.06% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 84 files with indirect coverage changes

ℹ️ Need help interpreting these results?

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.

1 participant