Skip to content

Conversation

ridouku
Copy link
Contributor

@ridouku ridouku commented Aug 13, 2021

Description

Currently, the project works without pinned dependencies, therefore there are inconsistencies between environments and troubles when a merge between a new branch with master is required the package-lock.json has many inconsistencies because the dependencies has caret or tilde symbol in their version and this allows to install patch o new features.

Solution

This PR modified the package.json to pin dependencies and prevent the inconsistencies, some dependencies were not change to static version and is required a future revision.

@codecov
Copy link

codecov bot commented Aug 13, 2021

Codecov Report

Merging #721 (9a68415) into master (13de42e) will increase coverage by 2.14%.
The diff coverage is 96.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #721      +/-   ##
==========================================
+ Coverage   93.09%   95.24%   +2.14%     
==========================================
  Files          85      106      +21     
  Lines        1564     2206     +642     
  Branches      107      169      +62     
==========================================
+ Hits         1456     2101     +645     
+ Misses         67       61       -6     
- Partials       41       44       +3     
Impacted Files Coverage Δ
...nents/time-range-form/time-range-form.component.ts 100.00% <ø> (ø)
...time-entries-table/time-entries-table.component.ts 63.63% <60.00%> (+8.08%) ⬆️
...toggles/filters/feature-filter-provider.service.ts 80.00% <62.50%> (+2.22%) ⬆️
src/app/modules/user/store/user.reducer.ts 62.50% <62.50%> (ø)
...ared/components/input-date/input-date.component.ts 66.66% <66.66%> (-2.57%) ⬇️
...dules/time-entries/pages/time-entries.component.ts 88.00% <90.00%> (+5.18%) ⬆️
...project-list-hover/project-list-hover.component.ts 90.38% <92.00%> (+2.01%) ⬆️
.../components/project-list/project-list.component.ts 96.66% <93.33%> (+12.45%) ⬆️
src/app/modules/users/store/user.reducers.ts 94.73% <94.73%> (ø)
...-entries/components/calendar/calendar.component.ts 95.16% <95.16%> (ø)
... and 84 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 b135a96...9a68415. Read the comment docs.

edgardavid2015 and others added 2 commits August 13, 2021 12:09
* fix: TT-304 Handle message: the data could not be load

* test: TT-304 Creating an unit test to coverage the new feature

Co-authored-by: Edgar Guaman <[email protected]>
ridouku and others added 6 commits August 16, 2021 12:02
…me-tracker-ui into TT-309-Implement-static-packages
* fix: TT-311 Can not update entries fixed

* code-smell: TT-311 Code smells suggested by Sonarcloud fixed

* fix: TT-311 Fix tests that fails on different timezone

* fix: TT-311 Spelling error on method fixed
…me-tracker-ui into TT-309-Implement-static-packages
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ridouku ridouku merged commit 87b2053 into master Aug 16, 2021
@ridouku ridouku deleted the TT-309-Implement-static-packages branch August 16, 2021 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants