Skip to content

3.7.1 Updated to gulp.js 4.0.x - #840

Closed
warengonzaga wants to merge 42 commits into
animate-css:masterfrom
warengonzaga:master
Closed

3.7.1 Updated to gulp.js 4.0.x#840
warengonzaga wants to merge 42 commits into
animate-css:masterfrom
warengonzaga:master

Conversation

@warengonzaga

@warengonzaga warengonzaga commented Jul 24, 2018

Copy link
Copy Markdown
Member

Hello, @daneden I manage to fix the conflicts... I am confused with the rebase where should I start. So I get the latest copy from you guys by using upstream. So let me know if this pull request is good?

  • Minor Bug Fixes
  • Updated Builds to 3.7.1
  • Migrated from gulp.js 3.9.x to gulp.js 4.0.x

@warengonzaga warengonzaga changed the title 3.7.0 Conflicts Fixed Update Builds 3.7.0 Jul 24, 2018
@daneden

daneden commented Jul 24, 2018

Copy link
Copy Markdown
Collaborator

I don’t think there's any action required here actually @warengonzaga: I commented on #834 mentioning that your rebase to master seemed to fix conflicts and everything is working fine now. Looks like this PR just updates package-lock.json, which I don’t think is necessary here and I'd rather not cause thrash with another minor release.

@daneden daneden closed this Jul 24, 2018
@warengonzaga warengonzaga changed the title Update Builds 3.7.0 3.7.1 Updated to gulp.js 4.0.x Jul 24, 2018
@daneden

daneden commented Jul 24, 2018

Copy link
Copy Markdown
Collaborator

@warengonzaga additionally, for future PRs you might consider forking again or updating from this repo's master branch since it looks like your pull requests are bringing a ton of old commits with them!

@warengonzaga

Copy link
Copy Markdown
Member Author

@daneden can you please include in the 3.7.0 release that we are already in GULP.JS 4 :) you forgot it.

@warengonzaga

Copy link
Copy Markdown
Member Author

Alright, @daneden will do it...

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