Skip to content
This repository was archived by the owner on Aug 31, 2023. It is now read-only.

fix: broken ci #2989

Merged
merged 1 commit into from
Aug 2, 2022
Merged

fix: broken ci #2989

merged 1 commit into from
Aug 2, 2022

Conversation

IWANABETHATGUY
Copy link
Contributor

@IWANABETHATGUY IWANABETHATGUY commented Aug 2, 2022 •
edited
Loading

Summary

  1. Disable the case that has syntax error due to broken CI
  2. Diable for now, recover the case later if we have solution to testing analyzer that have syntax error.

Test Plan

  1. CI should pass.

@ematipico
Copy link
Contributor

Where did we introduce this error?

@IWANABETHATGUY
Copy link
Contributor Author

Where did we introduce this error?

#2954 here,
I want to verify the unexpected code action is no more exist, so I add this test case that has syntax error.

@ematipico ematipico merged commit 9fb6a7c into rome:main Aug 2, 2022
@IWANABETHATGUY IWANABETHATGUY deleted the ci/fix-ci branch August 2, 2022 11:46
IWANABETHATGUY added a commit to IWANABETHATGUY/tools that referenced this pull request Aug 22, 2022
@IWANABETHATGUY
fix: broken ci (rome#2989)
0970008
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ematipico ematipico ematipico approved these changes

@leops leops Awaiting requested review from leops

@xunilrj xunilrj Awaiting requested review from xunilrj

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@IWANABETHATGUY @ematipico