Skip to content

Conversation

@rizwanjiwan
Copy link

Causing test to fail and doesn't add anything. bostore_contest.id is already in there, unique, and primary key.
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.97%. Comparing base (3417fea) to head (2951a74).
Report is 4 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

❗ There is a different number of reports uploaded between BASE (3417fea) and HEAD (2951a74). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (3417fea) HEAD (2951a74)
7.4 4 3
5-max 4 3
agnostic 1 0
Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2016       +/-   ##
=============================================
- Coverage     89.25%   73.97%   -15.28%     
  Complexity     8065     8065               
=============================================
  Files           232      232               
  Lines         24550    24550               
=============================================
- Hits          21911    18161     -3750     
- Misses         2639     6389     +3750     
Flag Coverage Δ
5-max 73.97% <ø> (-15.28%) ⬇️
7.4 73.97% <ø> (-15.28%) ⬇️
agnostic ?
mysql 69.79% <ø> (-0.02%) ⬇️
pgsql 69.84% <ø> (ø)
sqlite 67.79% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@smhg
Copy link
Contributor

smhg commented Jun 17, 2025

@rizwanjiwan would you be willing to reopen this PR?
The single change to Propel\Generator\Builder\Om\QueryBuilder seems useful.

If so, please rebase on master and remove the other changes. CI will now run the testsuite correctly so hopefully we can merge it afterwards.

Thank you!

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