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

Feature: Custom favicon#463

Merged
juanpicado merged 6 commits intomasterfrom
feat/custom-favicon
May 17, 2020
Merged

Feature: Custom favicon#463
juanpicado merged 6 commits intomasterfrom
feat/custom-favicon

Conversation

@AvailCat
Copy link
Copy Markdown
Contributor

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 28, 2020

Codecov Report

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

@@            Coverage Diff             @@
##           master     #463      +/-   ##
==========================================
+ Coverage   82.74%   82.82%   +0.07%     
==========================================
  Files         142      142              
  Lines        1084     1083       -1     
  Branches      222      210      -12     
==========================================
  Hits          897      897              
  Misses        154      154              
+ Partials       33       32       -1     
Impacted Files Coverage Δ
i18n/config.ts 100.00% <0.00%> (ø)
src/components/Icon/Icon.tsx 100.00% <0.00%> (ø)
src/components/Icon/styles.ts 88.88% <0.00%> (ø)
src/components/LanguageSwitch/LanguageSwitch.tsx 55.55% <0.00%> (+1.01%) ⬆️

@AvailCat AvailCat requested a review from juanpicado April 28, 2020 15:56
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.

👍 all good, but don't merge until other PR are being merged.

Copy link
Copy Markdown
Contributor

@priscilawebdev priscilawebdev left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@verdacciobot
Copy link
Copy Markdown

Thanks for your PR, the @verdaccio/ui package will be accessible from here for testing purposes:

npm install @verdaccio/[email protected] --registry https://registry.verdaccio.org

@juanpicado juanpicado merged commit 4713dea into master May 17, 2020
@delete-merged-branch delete-merged-branch bot deleted the feat/custom-favicon branch May 17, 2020 11:41
juanpicado added a commit that referenced this pull request May 29, 2020
This reverts commit 4713dea to unblock deployments
juanpicado added a commit that referenced this pull request May 31, 2020
This reverts commit 4713dea to unblock deployments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants