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

fix(@schematics/angular): align differential loading migration to br… #14226

Merged

Conversation

emilio-martinez
Copy link
Contributor

…owserslist template

Modifies differential loading migration script to align with the changes in 4d7581d to support Googlebot in browserslist by default.

Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @emilio-martinez ,

Thanks for this, can you please fix the failing tests and also change the commit to a fix?

…wserslist template

Modifies differential loading migration script to align with the changes in 4d7581d to support Googlebot in browserslist by default.
@emilio-martinez emilio-martinez force-pushed the fix/diff-loading-migration-content branch from 9a0c254 to abbc4a5 Compare April 23, 2019 03:32
@emilio-martinez emilio-martinez changed the title feat(@schematics/angular): align differential loading migration to br… fix(@schematics/angular): align differential loading migration to br… Apr 23, 2019
@emilio-martinez
Copy link
Contributor Author

@alan-agius4 Sure thing on both counts. Ready for your review.

Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this.
LGTM

@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Apr 23, 2019
not dead
not IE 9-11 # For IE 9-11 support, remove 'not'.
not Chrome 41 # For Googlebot support, remove 'not'.`;
not IE 9-11 # For IE 9-11 support, remove 'not'.`;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some strange stray characters here. I'll merge anyway to unblock release, could fix in a follow-up?

@alexeagle alexeagle merged commit 5f44e64 into angular:master Apr 23, 2019
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants