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

Redisable require.ensure() #3121

Merged
merged 1 commit into from
Jan 10, 2018

Conversation

everdimension
Copy link
Contributor

As this (#2176) issue has been closed, seems like this code was meant to be brought back.

@Timer
Copy link
Contributor

Timer commented Sep 13, 2017

The accompanying eslint rule will need re-enabled too. 😄

@Timer Timer added this to the 2.0.0 milestone Sep 13, 2017
@everdimension
Copy link
Contributor Author

Oh, right! One moment

@everdimension everdimension force-pushed the redisable-require-ensure branch from adaeb5b to 315d962 Compare September 13, 2017 23:06
@everdimension
Copy link
Contributor Author

Done!

@Timer Timer requested review from gaearon and removed request for gaearon September 14, 2017 00:29
@Timer
Copy link
Contributor

Timer commented Sep 14, 2017

Sigh, sorry for requesting a review by accident @gaearon; this is the 2nd time I've done this. 😛

@Timer
Copy link
Contributor

Timer commented Sep 14, 2017

Thanks @everdimension! This will probably be on the back burner until 2.0.0. 😄

@react-scripts-dangerous

Hello! I'm a bot that helps facilitate testing pull requests.

Your pull request (commit 315d962) has been released on npm for testing purposes.

npm i react-scripts-canary@1.0.14-315d962.0
# or
yarn add react-scripts-canary@1.0.14-315d962.0
# or
create-react-app --scripts-version=react-scripts-canary@1.0.14-315d962.0 folder/

Thanks for your contribution!

@Timer Timer changed the base branch from master to next January 10, 2018 01:38
@Timer
Copy link
Contributor

Timer commented Jan 10, 2018

We might as well try this again. Better now that there's the import hints.
/cc @gaearon

@Timer Timer merged commit 38db3df into facebook:next Jan 10, 2018
gaearon pushed a commit that referenced this pull request Jan 10, 2018
Timer pushed a commit to Timer/create-react-app that referenced this pull request Jan 11, 2018
Timer pushed a commit that referenced this pull request Jan 11, 2018
Timer pushed a commit to Timer/create-react-app that referenced this pull request Jan 11, 2018
Timer pushed a commit to Timer/create-react-app that referenced this pull request Jan 13, 2018
gaearon pushed a commit that referenced this pull request Jan 13, 2018
Timer pushed a commit that referenced this pull request Jan 14, 2018
gaearon pushed a commit that referenced this pull request Jan 14, 2018
gaearon pushed a commit that referenced this pull request Jan 14, 2018
gaearon pushed a commit that referenced this pull request Jan 14, 2018
gaearon pushed a commit that referenced this pull request Jan 14, 2018
Timer pushed a commit to Timer/create-react-app that referenced this pull request Jan 15, 2018
akstuhl pushed a commit to akstuhl/create-react-app that referenced this pull request Mar 15, 2018
zmitry pushed a commit to zmitry/create-react-app that referenced this pull request Sep 30, 2018
@lock lock bot locked and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants