Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[project-base] upgrade npm dependencies #1668

Merged
merged 3 commits into from
Feb 18, 2020
Merged

Conversation

pesektomas
Copy link
Contributor

@pesektomas pesektomas commented Feb 7, 2020

Q A
Description, reason for the PR After run npm audit we have 53 vulnerabilities. If we updated some dev dependencies we can have 0 vulnerabilities. This PR updated dev dependencies.
New feature No
BC breaks No
Fixes issues closes #756
Have you read and signed our License Agreement for contributions? Yes

@pesektomas pesektomas force-pushed the tp-upgrade-npm-dependencies branch 3 times, most recently from 60f5d24 to 2553d50 Compare February 12, 2020 06:56
@TomasGottvald
Copy link
Contributor

After regeneration of svg font - there is another style of closing cross (there is outline)- I think, we should change old svg icon for new one and it will be fixed -
obrazek

@TomasGottvald
Copy link
Contributor

I just pushed new icon which looks much better.
obrazek

@grossmannmartin grossmannmartin force-pushed the tp-upgrade-npm-dependencies branch from bd67aa8 to 4d8064e Compare February 18, 2020 11:14
@sonarqubecloud
Copy link

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
No Duplication information No Duplication information

@pesektomas pesektomas merged commit 95f3fed into 7.3 Feb 18, 2020
@pesektomas pesektomas deleted the tp-upgrade-npm-dependencies branch April 24, 2020 06:41
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.

3 participants