Skip to content
This repository was archived by the owner on Jun 1, 2025. It is now read-only.

feat(euro): add Euro Formatters and Sorters (day/month/year) #190

Merged
merged 1 commit into from
May 17, 2019

Conversation

ghiscoding
Copy link
Owner

  • ref #185

@codecov-io
Copy link

Codecov Report

Merging #190 into master will increase coverage by 0.5%.
The diff coverage is 66.12%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #190     +/-   ##
=========================================
+ Coverage   38.36%   38.86%   +0.5%     
=========================================
  Files         150      158      +8     
  Lines        6461     6579    +118     
  Branches     1941     2031     +90     
=========================================
+ Hits         2478     2556     +78     
- Misses       3974     4013     +39     
- Partials        9       10      +1
Impacted Files Coverage Δ
...dules/angular-slickgrid/sorters/sorterUtilities.ts 40% <0%> (-7.61%) ⬇️
src/app/modules/angular-slickgrid/sorters/index.ts 100% <100%> (ø) ⬆️
...-slickgrid/formatters/dateTimeEuroAmPmFormatter.ts 100% <100%> (ø)
...s/angular-slickgrid/sorters/dateEuroShortSorter.ts 100% <100%> (ø)
.../angular-slickgrid/formatters/dateEuroFormatter.ts 100% <100%> (ø)
.../app/modules/angular-slickgrid/formatters/index.ts 100% <100%> (ø) ⬆️
...ular-slickgrid/formatters/dateTimeEuroFormatter.ts 100% <100%> (ø)
...slickgrid/formatters/dateTimeShortEuroFormatter.ts 100% <100%> (ø)
...odules/angular-slickgrid/sorters/dateEuroSorter.ts 100% <100%> (ø)
...dules/angular-slickgrid/filter-conditions/index.ts 100% <100%> (ø) ⬆️
... and 13 more

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 dafc529...cdbc43a. Read the comment docs.

@ghiscoding ghiscoding merged commit 547277d into master May 17, 2019
@ghiscoding ghiscoding deleted the feat/date-euro-format branch May 17, 2019 14:18
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