Skip to content
This repository was archived by the owner on Sep 3, 2022. It is now read-only.

Conversation

danieljackins
Copy link
Contributor

@danieljackins danieljackins commented Sep 30, 2021

Description

Checklist

  • Thorough explanation of the issue/solution, and a link to the related issue
  • CI tests are passing
  • Unit tests were written for any new code
  • Code coverage is at least maintained, or increased.

@codecov-commenter
Copy link

Codecov Report

Merging #258 (1a5dcc0) into master (42f410f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #258   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            3         3           
=========================================
  Hits             3         3           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 42f410f...1a5dcc0. Read the comment docs.

Copy link
Contributor

@juliofarah juliofarah left a comment

Choose a reason for hiding this comment

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

🥳 🎉

Wanna add some details to the PR description?

@danieljackins danieljackins merged commit 7558b32 into master Sep 30, 2021
@danieljackins danieljackins deleted the reference-next branch September 30, 2021 17:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants