Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wip] Grunt -> Gulp #5267

Closed
wants to merge 9 commits into from
Closed

[wip] Grunt -> Gulp #5267

wants to merge 9 commits into from

Conversation

zertosh
Copy link
Contributor

@zertosh zertosh commented Oct 24, 2015

Fixes #4927

@facebook-github-bot
Copy link

@zertosh updated the pull request.

@facebook-github-bot
Copy link

@zertosh updated the pull request.

@facebook-github-bot
Copy link

@zertosh updated the pull request.

@facebook-github-bot
Copy link

@zertosh updated the pull request.

@zpao
Copy link
Member

zpao commented Oct 26, 2015

cc @nickdima.

@zertosh - is this something you're going to continue working on? I think @nickdima had also started so I'd like to make sure we aren't wasting anybody's time…

@zertosh
Copy link
Contributor Author

zertosh commented Oct 26, 2015

@zpao, this is the first pass, where it's mostly a 1:1 grunt to gulp port. the 2nd pass will be to improve things once they're in gulp (e.g. not spawning processes for linting, jest, etc... reuse result streams instead of reading from disk). I worked on this this weekend, and I can wrap it up next week. Not sure how much @nickdima has already.

@nickdima
Copy link

I didn't have too much time yet to actually implement something, just figuring out everything that's going on with the current build process alone took quite some time. :)
@zertosh your approach sounds good. I'll follow this PR in case I can help with anything.

@jimfb
Copy link
Contributor

jimfb commented Jan 8, 2016

No activity in a couple of months, all checks failed, merge conflicts. I'm going to close this out. We can re-examine if/when the PR is in a better state.

@jimfb jimfb closed this Jan 8, 2016
@zertosh
Copy link
Contributor Author

zertosh commented Jan 8, 2016

😊 totally forgot about this PR... i don't want to abandon it. i'll open a new one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants