Skip to content
This repository has been archived by the owner on Jan 16, 2022. It is now read-only.

refactor: adds no uplink spec #213

Merged
merged 2 commits into from Oct 25, 2019
Merged

Conversation

ayusharma
Copy link
Member

Type: Unit test

The following has been addressed in the PR:

  • There is a related issue? No
  • Unit or Functional tests are included in the PR > Yes

Description:

  • Adds missing no uplink spec

@codecov
Copy link

codecov bot commented Oct 24, 2019

Codecov Report

Merging #213 into master will increase coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
+ Coverage   87.41%   87.52%   +0.11%     
==========================================
  Files         117      117              
  Lines         874      874              
  Branches      139      160      +21     
==========================================
+ Hits          764      765       +1     
+ Misses         98       94       -4     
- Partials       12       15       +3
Impacted Files Coverage Δ
src/utils/styles/media.ts 81.81% <0%> (ø) ⬆️
src/components/DetailSidebar/DetailSidebar.tsx 85% <0%> (ø) ⬆️
src/components/UpLinks/UpLinks.tsx 100% <0%> (+11.11%) ⬆️

Copy link
Member

@juanpicado juanpicado left a comment

Choose a reason for hiding this comment

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

👍 welcome 🙃

@juanpicado juanpicado merged commit ade548a into master Oct 25, 2019
@delete-merged-branch delete-merged-branch bot deleted the adds-missing-no-uplink-spec branch October 25, 2019 18:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants