Skip to content

test: Disable E2E tests#219

Merged
lforst merged 2 commits intomainfrom
lforst-disable-e2e-tests
Apr 28, 2023
Merged

test: Disable E2E tests#219
lforst merged 2 commits intomainfrom
lforst-disable-e2e-tests

Conversation

@lforst
Copy link
Copy Markdown

@lforst lforst commented Apr 28, 2023

The switch to the newer artifact bundle upload has rendered our e2e tests faulty because the endpoint we ping to check for uploaded source maps doesn't return them anymore.

For now we disable them so we can have green CI again.

@lforst lforst requested a review from Lms24 April 28, 2023 09:31
Comment thread .github/workflows/checks.yml Outdated
(github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository) &&
github.actor != 'dependabot[bot]'
github.actor != 'dependabot[bot]' &&
${{ false }} # Debug ID backend changes broke the E2E tests, we need to revisit them
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

not sure if this will work, nesting this inside of if here? I think the safer is 1 == 2 or something like this? Never 100% sure about this in the GHA yaml format 😅

@lforst lforst merged commit 87ca5ef into main Apr 28, 2023
@lforst lforst deleted the lforst-disable-e2e-tests branch April 28, 2023 11:27
@lforst lforst mentioned this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants