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

Fix: removed unused style file #406

Merged
merged 1 commit into from Jan 2, 2020

Conversation

priscilawebdev
Copy link
Contributor

Type: Bug

Description:

Fixes #95

@priscilawebdev priscilawebdev self-assigned this Jan 2, 2020
@priscilawebdev priscilawebdev added the invalid This doesn't seem right label Jan 2, 2020
@sergiohgz
Copy link
Member

Did you add invalid for a reason?

@priscilawebdev
Copy link
Contributor Author

priscilawebdev commented Jan 2, 2020

@sergiohgz yes... the code I removed doesn't seems right..hmm but actually it is the solution...I will remove the tag :)

@priscilawebdev priscilawebdev removed the invalid This doesn't seem right label Jan 2, 2020
@codecov
Copy link

codecov bot commented Jan 2, 2020

Codecov Report

Merging #406 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #406   +/-   ##
=======================================
  Coverage   87.72%   87.72%           
=======================================
  Files         137      137           
  Lines         953      953           
  Branches      211      198   -13     
=======================================
  Hits          836      836           
- Misses        103      104    +1     
+ Partials       14       13    -1
Impacted Files Coverage Δ
src/components/Icon/styles.ts 88.88% <0%> (ø) ⬆️

@verdacciobot
Copy link

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

npm install @verdaccio/ui-theme@v0.3.11-27b0b30-pr406.0 --registry https://registry.verdaccio.org

@sergiohgz
Copy link
Member

If it doesn't seems right, does it really apply?

@priscilawebdev
Copy link
Contributor Author

priscilawebdev commented Jan 2, 2020

@sergiohgz sorry...I confused you..my PR solves the problem 😉the removed file was not used

@priscilawebdev priscilawebdev merged commit 6eeae63 into master Jan 2, 2020
@delete-merged-branch delete-merged-branch bot deleted the bug/95_removed_unused_style_file branch January 2, 2020 23:18
@sergiohgz
Copy link
Member

Aaaa, perfect 👌

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants