Skip to content

chore(deps): update gulp and bump deps#2201

Merged
dgutride merged 3 commits intopatternfly:masterfrom
redallen:chore/deps
Aug 29, 2019
Merged

chore(deps): update gulp and bump deps#2201
dgutride merged 3 commits intopatternfly:masterfrom
redallen:chore/deps

Conversation

@redallen
Copy link
Contributor

Closes #2039 . Node 10 compatible now.

We should consider replacing some of the gulp-* plugins with our own versions as they aren't that hard to implement in Gulp 4. This way we control the versions of node-sass or cssmin we use in building our CSS.

@patternfly-build
Copy link
Collaborator

PatternFly-Next preview: https://patternfly-next-pr-2201.surge.sh

@dgutride
Copy link
Member

nice job - what do you think about updating the engines values in package.json?

@redallen
Copy link
Contributor Author

I think we still support Node 8, we are just delivering on our promise of supporting node>=8 now, so it can remain unchanged unless we want to drop Node 8 support. I know folks still are using Node 8, though.

@redallen
Copy link
Contributor Author

redallen commented Aug 28, 2019

No diff in the min CSS comapred to @patternfly/patternfly 2.27.1:

➜  patternfly-react-seed git:(chore/text-input-example) ✗ diff node_modules/@patternfly/patternfly/patternfly.min.css ~/src/patternfly-next/dist/patternfly.min.css  
➜  patternfly-react-seed git:(chore/text-input-example) ✗ echo $?
0

@dgutride dgutride requested a review from mcoker August 28, 2019 19:05
Copy link
Contributor

@mcoker mcoker left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @redallen

@dgutride dgutride merged commit ed0017e into patternfly:master Aug 29, 2019
@redallen redallen deleted the chore/deps branch August 29, 2019 13:34
@redallen
Copy link
Contributor Author

🎉 This PR is included in version 2.27.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

4 participants