Skip to content

ci: upgrade to Sauce Connect 5 to fix SC4 deprecation - #34553

Merged
alan-agius4 merged 1 commit into
angular:21.2.xfrom
alan-agius4:ci/saucelabs-sc5-21.2.x
Oct 28, 2026
Merged

alan-agius4 merged 1 commit into
angular:21.2.xfrom
alan-agius4:ci/saucelabs-sc5-21.2.x

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

Upgrades the SauceLabs CI job on 21.2.x to use saucelabs/sauce-connect-action@v3 (Sauce Connect 5) matching main.

@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Oct 28, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request removes legacy Sauce Connect bash scripts and refactors the Protractor Sauce Labs configuration file to extract browser capabilities, set the region to 'us', and transition from using 'tunnelIdentifier' to 'tunnelName'. The feedback recommends conditionally including 'tunnelName' in the capabilities mapping to avoid serialization issues if the environment variable is undefined.

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: lts This PR is targeting a version currently in long-term support labels Oct 28, 2026
@alan-agius4
alan-agius4 merged commit 77372ec into angular:21.2.x Oct 28, 2026
35 of 37 checks passed
@alan-agius4
alan-agius4 deleted the ci/saucelabs-sc5-21.2.x branch October 28, 2026 09:14
@alan-agius4

Copy link
Copy Markdown
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: lts This PR is targeting a version currently in long-term support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants