Skip to content

test: use correct snapshot versions in add-pwa E2E test with yarn#20956

Merged
clydin merged 1 commit intoangular:masterfrom
clydin:e2e/fix-add-pwa-snapshots
May 26, 2021
Merged

test: use correct snapshot versions in add-pwa E2E test with yarn#20956
clydin merged 1 commit intoangular:masterfrom
clydin:e2e/fix-add-pwa-snapshots

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented May 26, 2021

When executing the add-pwa E2E test with Angular snapshots, the package versions need to be updated after the ng add @angular/pwa call to ensure that the correct version of @angular/service-worker is used.
Yarn requires a different package installation strategy due to Yarn not always installing a new version of a github sha when directly adding an individual package.

When executing the add-pwa E2E test with Angular snapshots, the package versions need to be updated after the `ng add @angular/pwa` call to ensure that the correct version of `@angular/service-worker` is used.
Yarn requires a different package installation strategy due to Yarn not always installing a new version of a github sha when directly adding an individual package.
@clydin clydin added the target: minor This PR is targeted for the next minor release label May 26, 2021
@google-cla google-cla bot added the cla: yes label May 26, 2021
@clydin clydin requested a review from alan-agius4 May 26, 2021 14:46
@clydin clydin merged commit 37db1d2 into angular:master May 26, 2021
@clydin clydin deleted the e2e/fix-add-pwa-snapshots branch May 26, 2021 15:25
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Jun 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants