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(e2e): added conditial device usage #3886

Merged
merged 73 commits into from
Jan 9, 2025

Conversation

inc-man
Copy link
Collaborator

@inc-man inc-man commented Dec 6, 2024

Motivation

We want to ensure that macOS runners are ONLY used for Apple devices and that everything else is being tested on ubuntu runners.

Changes

adjusted the configuration for handling conditional device usage based on os.

@inc-man inc-man requested a review from a team as a code owner December 6, 2024 13:42
@AntonioVentilii AntonioVentilii added the run-e2e-snapshots It needs the CI for E2E Snapshots label Dec 6, 2024
@inc-man inc-man added the run-e2e-snapshots It needs the CI for E2E Snapshots label Dec 20, 2024
@inc-man inc-man removed the run-e2e-snapshots It needs the CI for E2E Snapshots label Dec 20, 2024
@inc-man inc-man added the run-e2e-snapshots It needs the CI for E2E Snapshots label Dec 20, 2024
@inc-man inc-man removed the run-e2e-snapshots It needs the CI for E2E Snapshots label Dec 20, 2024
AntonioVentilii added a commit that referenced this pull request Jan 6, 2025
# Motivation

Splitting #3886 , we first
define the MATRIX_OS environment variable for each E2E os.

---------

Co-authored-by: Nico Mächler <[email protected]>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
@inc-man inc-man added the run-e2e-snapshots It needs the CI for E2E Snapshots label Jan 9, 2025
@inc-man inc-man merged commit 7a7f15d into main Jan 9, 2025
23 checks passed
@inc-man inc-man deleted the fix(e2e)/include-devices-based-on-os branch January 9, 2025 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-e2e-snapshots It needs the CI for E2E Snapshots
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants