Skip to content

test: use correct snapshot versions in add-pwa E2E test#20925

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

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

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented May 24, 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.

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.
@clydin clydin added the target: patch This PR is targeted for the next patch release label May 24, 2021
@clydin clydin requested a review from alan-agius4 May 24, 2021 21:13
@google-cla google-cla bot added the cla: yes label May 24, 2021
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label May 25, 2021
@clydin clydin added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels May 25, 2021
@clydin clydin merged commit c44100b into angular:master May 25, 2021
@clydin clydin deleted the e2e/fix-add-pwa-snapshots branch May 25, 2021 13:54
@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 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker 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