Skip to content
This repository has been archived by the owner on Aug 5, 2021. It is now read-only.

SONAR-13155 Upgrade dependencies #101

Merged
merged 2 commits into from Mar 19, 2020

Conversation

jeremy-davis-sonarsource
Copy link
Contributor

@jeremy-davis-sonarsource jeremy-davis-sonarsource commented Mar 18, 2020

Goes with SONAR#2615

/!\ SC will have to align some dependency versions for this to work!

Checklist

  • Functional validation
  • Update changelog

@@ -39,8 +39,8 @@ exports[`should render correctly 1`] = `
className="line-chart-grid"
x1={0}
x2={30}
y1={34.666666666666664}
y2={34.666666666666664}
y1={34.66666666666667}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting... 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My guess is that it's d3/d3-interpolate#73 ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, even if the +0.0000000004 on every value is strange.

@sonarsource-next
Copy link

Kudos, SonarQube 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

@jeremy-davis-sonarsource jeremy-davis-sonarsource merged commit f956131 into master Mar 19, 2020
@jeremy-davis-sonarsource jeremy-davis-sonarsource deleted the feature/jay/upgrade-dependencies branch March 19, 2020 13:41
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.

None yet

2 participants