Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

chore(share/getters): add unit test for ctxwithsplittimeout #2374

Merged
merged 4 commits into from
Jun 21, 2023

Conversation

walldiss
Copy link
Member

Overview

Adds extra safety.

@walldiss walldiss added the kind:testing Related to unit tests label Jun 16, 2023
@walldiss walldiss requested a review from renaynay as a code owner June 16, 2023 10:41
@walldiss walldiss self-assigned this Jun 16, 2023
distractedm1nd
distractedm1nd previously approved these changes Jun 16, 2023
Copy link
Collaborator

@distractedm1nd distractedm1nd left a comment

Choose a reason for hiding this comment

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

Thank you for this

Wondertan
Wondertan previously approved these changes Jun 16, 2023
Copy link
Member

@Wondertan Wondertan left a comment

Choose a reason for hiding this comment

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

❤️

@codecov-commenter
Copy link

Codecov Report

Merging #2374 (bf8aa40) into main (fd18184) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2374      +/-   ##
==========================================
- Coverage   50.72%   50.70%   -0.03%     
==========================================
  Files         155      155              
  Lines        9830     9830              
==========================================
- Hits         4986     4984       -2     
- Misses       4407     4410       +3     
+ Partials      437      436       -1     

see 3 files with indirect coverage changes

distractedm1nd
distractedm1nd previously approved these changes Jun 16, 2023
Wondertan
Wondertan previously approved these changes Jun 16, 2023
renaynay
renaynay previously approved these changes Jun 16, 2023
@walldiss walldiss enabled auto-merge June 16, 2023 14:43
auto-merge was automatically disabled June 21, 2023 13:28

Merge queue setting changed

@Wondertan Wondertan enabled auto-merge June 21, 2023 15:35
@Wondertan Wondertan added this pull request to the merge queue Jun 21, 2023
Merged via the queue into celestiaorg:main with commit 367fc66 Jun 21, 2023
11 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:testing Related to unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants