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 #186

Closed
wants to merge 1 commit into from
Closed

fix(compat): improve react version detection #186

wants to merge 1 commit into from

Conversation

acgrdumlu
Copy link
Contributor

@acgrdumlu acgrdumlu commented Jun 16, 2017

What:

Why:

How:

Checklist:

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

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.

Looks great! Two things:

  1. update the code to be parseFloat instead of ParseFloat (my bad)
  2. This is optional, but you can add yourself to the contributors table. Run npm start contributors.add

Thanks!

@@ -3,7 +3,7 @@ import React from 'react'
let PropTypes

/* istanbul ignore next */
if (React.version.slice(0, 4) === '15.5') {
if (ParseFloat(React.version.slice(0, 4)) >= 15.5) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Whoops! This should be a lower-case p. Sorry about that! So it should be:

if (parseFloat(React.version.slice(0, 4)) >= 15.5) {

@kentcdodds
Copy link
Contributor

Oh, and another bit of feedback, normally when making a pull request, you'll want to fill in the Pull Request description with something useful. Some open source projects (like this one) will even have a template for you to fill out.

@acgrdumlu
Copy link
Contributor Author

acgrdumlu commented Jun 16, 2017

Hi @kentcdodds
I cannot push again after the changes that you want me to.
When I push the codes with this:
git push --set-upstream origin pr/fix-react-version-detection-new
I'm giving this error;
remote: Permission to paypal/glamorous.git denied to acgrdumlu.
fatal: unable to access 'https://github.com/paypal/glamorous.git/': The requested URL returned error: 403

Could you help me about this issue?

@kentcdodds
Copy link
Contributor

cc @vesparny (or anyone else), could you help @acgrdumlu? I'm afraid I don't have time (getting ready for family vacation!)

@atticoos
Copy link
Collaborator

Make sure you push to your fork, @acgrdumlu. Sounds like you may be attempting to push to paypal/glamorous

@kwelch
Copy link
Contributor

kwelch commented Jun 17, 2017

This is now resolved with the merge of #189, right?

@vesparny
Copy link
Contributor

Yep. Closing

@vesparny vesparny closed this Jun 17, 2017
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.

5 participants