Skip to content

Conversation

@trexfeathers
Copy link
Contributor

@trexfeathers trexfeathers commented Mar 18, 2024

Does not need to be a merge-commit since this is a 'manual' recreation of what the merge-back should be (no features are exclusive to the release branch, only What's New content).

@trexfeathers trexfeathers added the Type: Merge Back Merging a feature branch back to main label Mar 18, 2024
@trexfeathers trexfeathers requested a review from bjlittle March 18, 2024 16:50
@trexfeathers trexfeathers mentioned this pull request Mar 18, 2024
@codecov
Copy link

codecov bot commented Mar 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.74%. Comparing base (93d7d4a) to head (ae61baf).
Report is 118 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5857      +/-   ##
==========================================
+ Coverage   89.37%   89.74%   +0.37%     
==========================================
  Files          89       92       +3     
  Lines       22444    22942     +498     
  Branches     5382     5464      +82     
==========================================
+ Hits        20059    20590     +531     
+ Misses       1639     1620      -19     
+ Partials      746      732      -14     

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

@bjlittle bjlittle self-assigned this Mar 19, 2024
Copy link
Member

@bjlittle bjlittle left a comment

Choose a reason for hiding this comment

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

@trexfeathers LGTM

Thanks for going this, nice one 👍

@bjlittle bjlittle merged commit 3b49086 into SciTools:main Mar 19, 2024
@trexfeathers trexfeathers deleted the v3.7.1.mergeback branch May 3, 2024 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Merge Back Merging a feature branch back to main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants