search

dotnet / nuget-trends

Check out NuGet packages adoption and what's trending on NuGet.
https://nugettrends.com
MIT License
146 stars 25 forks source link

Truncate white-space and give feedback if no result for search #123

Closed frosch95 closed 3 years ago

frosch95 commented 3 years ago

solves #63

dnfadmin commented 3 years ago

CLA assistant check
All CLA requirements met.

codecov[bot] commented 3 years ago

Codecov Report

Merging #123 into main will decrease coverage by 1.31%. The diff coverage is 61.53%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
- Coverage   91.36%   90.05%   -1.32%     
==========================================
  Files          28       28              
  Lines         336      342       +6     
  Branches       28       30       +2     
==========================================
+ Hits          307      308       +1     
- Misses         22       26       +4     
- Partials        7        8       +1
Impacted Files Coverage Δ
src/NuGetTrends.Spa/src/app/mocks/router-mock.ts 100.00% <ø> (ø)
src/NuGetTrends.Spa/src/app/mocks/toaster-mock.ts 100.00% <ø> (ø)
.../components/search-input/search-input.component.ts 82.35% <58.33%> (-8.35%) :arrow_down:
...Trends.Spa/src/app/core/footer/footer.component.ts 100.00% <100.00%> (ø)
...ed/components/search-type/search-type.component.ts 100.00% <0.00%> (ø)
...omponents/share-popover/share-popover.component.ts 100.00% <0.00%> (ø)

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 2595477...164a9c1. Read the comment docs.

joaopgrassi commented 3 years ago

Thanks for the help!

frosch95 commented 3 years ago

You are welcome!