Skip to content

Conversation

ST-DDT
Copy link
Member

@ST-DDT ST-DDT commented Oct 7, 2023

@ST-DDT ST-DDT added p: 1-normal Nothing urgent c: infra Changes to our infrastructure or project setup labels Oct 7, 2023
@ST-DDT ST-DDT added this to the vAnytime milestone Oct 7, 2023
@ST-DDT ST-DDT requested review from a team October 7, 2023 13:50
@ST-DDT ST-DDT self-assigned this Oct 7, 2023
@codecov
Copy link

codecov bot commented Oct 7, 2023

Codecov Report

Merging #2443 (8d7d542) into next (eaf714d) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #2443   +/-   ##
=======================================
  Coverage   99.58%   99.59%           
=======================================
  Files        2823     2823           
  Lines      255523   255523           
  Branches     1102     1105    +3     
=======================================
+ Hits       254470   254478    +8     
+ Misses       1025     1017    -8     
  Partials       28       28           

see 1 file with indirect coverage changes

Copy link
Member

@xDivisionByZerox xDivisionByZerox left a comment

Choose a reason for hiding this comment

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

Makes sense to get rid of the "unreadable" reduce functions, I guess. No strong feelings about this.

@ST-DDT ST-DDT enabled auto-merge (squash) October 8, 2023 15:18
@ST-DDT ST-DDT merged commit 3e4952b into next Oct 8, 2023
@xDivisionByZerox xDivisionByZerox deleted the infra/unicorn/prefer-object-from-entries branch October 8, 2023 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: infra Changes to our infrastructure or project setup p: 1-normal Nothing urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants