Skip to content
This repository has been archived by the owner on Jul 7, 2022. It is now read-only.

Temporarily disable AR screen test #386

Merged
merged 1 commit into from
May 16, 2022
Merged

Conversation

alexandrepiveteau
Copy link
Contributor

@alexandrepiveteau alexandrepiveteau commented May 16, 2022

Since #213 makes many runs on the CI fail, this PR temporarily disables this flaky test (until a proper solution is found).

@alexandrepiveteau alexandrepiveteau self-assigned this May 16, 2022
@alexandrepiveteau alexandrepiveteau added the update-me-please Right to update current branch with main label May 16, 2022
@alexandrepiveteau alexandrepiveteau marked this pull request as ready for review May 16, 2022 20:19
@alexandrepiveteau alexandrepiveteau enabled auto-merge (squash) May 16, 2022 20:21
@alexandrepiveteau alexandrepiveteau added the bug Something isn't working label May 16, 2022
Copy link
Contributor

@matt989253 matt989253 left a comment

Choose a reason for hiding this comment

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

Yes plz

@codeclimate
Copy link

codeclimate bot commented May 16, 2022

Code Climate has analyzed commit 813e06f and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 97.1% (-0.1% change).

View more on Code Climate.

Copy link
Contributor

@BadrTad BadrTad left a comment

Choose a reason for hiding this comment

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

Thanks

@alexandrepiveteau alexandrepiveteau merged commit 2ed085e into main May 16, 2022
@alexandrepiveteau alexandrepiveteau deleted the fix/ap/disable-ar-test branch May 16, 2022 21:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working update-me-please Right to update current branch with main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants