Skip to content

@types/prop-types#827

Merged
dahlbyk merged 6 commits into
masterfrom
@types/prop-types
Aug 12, 2018
Merged

@types/prop-types#827
dahlbyk merged 6 commits into
masterfrom
@types/prop-types

Conversation

@dahlbyk

@dahlbyk dahlbyk commented Aug 9, 2018

Copy link
Copy Markdown
Contributor

Griddle major version

1.13.1

Changes proposed in this pull request

Why these changes are made

Fixes #826

Are there tests?

Compiler!

@dahlbyk dahlbyk changed the title @types/prop types @types/prop-types Aug 9, 2018
@dahlbyk dahlbyk mentioned this pull request Aug 9, 2018
Comment thread .travis.yml
language: node_js
node_js:
- '5'
- '9'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Looks like build tooling has taken a dependency on object spread.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ahh yea we're a few versions behind on node there :D

Comment thread .travis.yml

branches:
only:
- master

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Minor pet peeve when Travis push and pr builds both run. 😀

@dahlbyk dahlbyk requested a review from ryanlanciaux August 9, 2018 20:37

@ryanlanciaux ryanlanciaux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good and I should clean up some things that were around for awhile as well as addresses the import with PropTypes.

@dahlbyk dahlbyk merged commit fc77b31 into master Aug 12, 2018
@dahlbyk dahlbyk deleted the @types/prop-types branch August 12, 2018 21:05
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.

2 participants