Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

fix(compat): improve react version detection #189

Merged
merged 1 commit into from
Jun 17, 2017
Merged

fix(compat): improve react version detection #189

merged 1 commit into from
Jun 17, 2017

Conversation

acgrdumlu
Copy link
Contributor

@acgrdumlu acgrdumlu commented Jun 17, 2017

What:
parseFloat() function added.

Why:
To make the code consistent.

How:
Adding parseFloat() to this line: (parseFloat(React.version.slice(0, 4)) >= 15.5)

Checklist:

  • Documentation
  • Tests
  • Code complete
  • Added myself to contributors table
  • Followed the commit message format

@codecov-io
Copy link

codecov-io commented Jun 17, 2017

Codecov Report

Merging #189 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #189   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          10     10           
  Lines         142    142           
  Branches       36     36           
=====================================
  Hits          142    142

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 f55f157...13a4d9d. Read the comment docs.

Copy link
Contributor

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

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

This is perfect! 🎉🎉🎉 Well done!

@kentcdodds kentcdodds merged commit 8ffffe2 into paypal:master Jun 17, 2017
@kentcdodds
Copy link
Contributor

Congratulations on your first ever pull request merge. Very well done. Thank you for your contribution!

@paulmolluzzo
Copy link
Collaborator

Congrats @acgrdumlu! 🎉 Keep it up!

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.

4 participants