Skip to content

Fixes unchecked access to 'deploy' script on build - #8292

Merged
andriijas merged 1 commit into
react:masterfrom
renato-bohler:master
Feb 8, 2020
Merged

Fixes unchecked access to 'deploy' script on build#8292
andriijas merged 1 commit into
react:masterfrom
renato-bohler:master

Conversation

@renato-bohler

Copy link
Copy Markdown
Contributor

Fixes#8291.

Tested executing PUBLIC_URL=http://renato-bohler.github.io/my-app yarn build on create-react-app root folder before and after changes.

Before changes (c03bb36)

image

After changes (fe7904a)

image

@stale

staleBot commented Feb 7, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had any recent activity. It will be closed in 5 days if no further activity occurs.

@stalestaleBot added the stale label Feb 7, 2020
@renato-bohler

Copy link
Copy Markdown
ContributorAuthor

Hmm, any thoughts @andriijas, @ianschmitz? I really don't think this should be closed.

@andriijas
andriijas merged commit 767aa18 into react:masterFeb 8, 2020
@andriijas

Copy link
Copy Markdown
Contributor

Thanks @renato-bohler

@locklockBot locked and limited conversation to collaborators Feb 13, 2020
@iansuiansu modified the milestones: 3.3.2, 3.4Feb 14, 2020
Sign up for freeto 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.

Build fails when package has no scripts

6 participants

@renato-bohler@andriijas@heyimalex@ianschmitz@iansu@facebook-github-bot