Skip to content

#765 - Work on removing publish feature flags - #861

Merged
gaearon merged 1 commit into
react:masterfrom
hycner:master
Oct 7, 2016
Merged

#765 - Work on removing publish feature flags#861
gaearon merged 1 commit into
react:masterfrom
hycner:master

Conversation

@hycner

Copy link
Copy Markdown
Contributor

Hi! Was poking around the repo and saw this #765 and thought I'd give it a shot.

I think this mostly works, but I've noticed a bit of destructive package.json touching during the process. Would love to get some feedback at this point and see if this is heading in the right direction.

Thanks!

@gaearon

Copy link
Copy Markdown
Contributor

Hi, thanks for PR but I think the issue explicitly says:

Claimed by @shubheksha

as the very first line.

@shubheksha I think you had some work in progress, could you push that?

@hycner

Copy link
Copy Markdown
ContributorAuthor

Hahaha I'm not sure how I missed that! Sorry was a very long day yesterday.

Ok well no worries if this has been claimed, then. I don't want to be stepping on any toes.

@hycnerhycner closed this Oct 6, 2016
@gaearon

Copy link
Copy Markdown
Contributor

Well, since you already implemented it, it doesn’t really make sense to implement it twice.
@shubheksha told me she didn’t advance far with it so let’s reopen.

But please read the issue carefully the next time.

@gaearongaearon reopened this Oct 7, 2016
@gaearongaearon added this to the 0.7.0 milestone Oct 7, 2016
@gaearon
gaearon merged commit 90ce8ab into react:masterOct 7, 2016
@gaearongaearon mentioned this pull request Oct 7, 2016
feiqitian pushed a commit to feiqitian/create-react-app that referenced this pull request Oct 25, 2016
jarlef pushed a commit to jarlef/create-react-app that referenced this pull request Nov 28, 2016
@locklockBot locked and limited conversation to collaborators Jan 22, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@hycner@gaearon@facebook-github-bot