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

[Feat]: added download tarball button at list#237

Merged
juanpicado merged 4 commits intoverdaccio:masterfrom
anikethsaha:small-feat/tarball-btn-at-list
Nov 7, 2019
Merged

[Feat]: added download tarball button at list#237
juanpicado merged 4 commits intoverdaccio:masterfrom
anikethsaha:small-feat/tarball-btn-at-list

Conversation

@anikethsaha
Copy link
Copy Markdown
Member

Type:
Feature

The following has been addressed in the PR:

Description:
Added the download package button in the package list from the main page

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 5, 2019

Codecov Report

Merging #237 into master will decrease coverage by 0.07%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
- Coverage   90.31%   90.23%   -0.08%     
==========================================
  Files         139      139              
  Lines         919      922       +3     
  Branches      142      161      +19     
==========================================
+ Hits          830      832       +2     
+ Misses         77       75       -2     
- Partials       12       15       +3
Impacted Files Coverage Δ
src/components/Package/Package.tsx 96.42% <66.66%> (-3.58%) ⬇️
src/utils/styles/media.ts 81.81% <0%> (ø) ⬆️
src/components/DetailSidebar/DetailSidebar.tsx 85% <0%> (ø) ⬆️

Copy link
Copy Markdown
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.

🚀 super nice !

Screen Shot 2019-11-07 at 6 04 12 PM

@juanpicado juanpicado merged commit bdef686 into verdaccio:master Nov 7, 2019
@juanpicado juanpicado mentioned this pull request Nov 8, 2019
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.

2 participants