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

Remove soft-fail marker from the CI tests #6971

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

mai93
Copy link
Collaborator

@mai93 mai93 commented Nov 6, 2024

it hides failures on the under-dev IDE versions and it is no longer needed since the compatibility changes for the new IDE versions are usually sent out in the same PR now

@github-actions github-actions bot added the awaiting-review Awaiting review from Bazel team on PRs label Nov 6, 2024
@mai93
Copy link
Collaborator Author

mai93 commented Nov 6, 2024

@LeFrosch can you check the CLion failures with 2024.3?

@tpasternak
Copy link
Collaborator

We need to wait until the next clion release - it will come with updated JS plugin

@ujohnny
Copy link
Collaborator

ujohnny commented Nov 7, 2024

From my point this was intentional, see #6856. I'm fine to keep under-dev builds failing, because it leaves us some space to work on it, but I guess right now we're just missed to update oldest and latest oss-stable refs.

@sgowroji sgowroji added the product: IntelliJ IntelliJ plugin label Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review Awaiting review from Bazel team on PRs product: IntelliJ IntelliJ plugin
Projects
Status: Untriaged
Development

Successfully merging this pull request may close these issues.

4 participants