Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Make -Prelease.overriddenBranchName work in non-detached state #887

Conversation

ddeath
Copy link
Contributor

@ddeath ddeath commented Feb 3, 2025

Make -Prelease.overriddenBranchName work in non-detached state

As per #885

Tested it locally and it worked.

@bgalek
Copy link
Member

bgalek commented Feb 24, 2025

LGTM, thx!

@bgalek bgalek enabled auto-merge (squash) February 24, 2025 13:05
Copy link

codecov bot commented Feb 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.96%. Comparing base (805f73a) to head (2545dc0).
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #887   +/-   ##
=========================================
  Coverage     61.96%   61.96%           
+ Complexity      442      441    -1     
=========================================
  Files            83       83           
  Lines          1680     1680           
  Branches        161      161           
=========================================
  Hits           1041     1041           
  Misses          570      570           
  Partials         69       69           

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

@bgalek bgalek merged commit 67a9818 into allegro:main Feb 24, 2025
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants