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

Enable required extended capabilities when loading project #486

Conversation

datho7561
Copy link
Contributor

@datho7561 datho7561 commented Feb 20, 2025

Fixes #450

TODO per-merge:

  • Remove GitHub Action

@datho7561
Copy link
Contributor Author

I threw in a GitHub Action to test that the changes work. I expect at least 1 failure, since the link in the Bulkhead docs is hardcoded to the Jenkins maven repo path

@datho7561
Copy link
Contributor Author

It seems the diagnostic tests are still flakey.

I think we need to make a decision if we are okay with testing the Bulkhead link knowing it will only work properly on Jenkins, or if we want to disable the extended capability for that test run and pretend the link doesn't exist.

Fixes eclipse-lsp4mp#450

Signed-off-by: David Thompson <davthomp@redhat.com>
@datho7561 datho7561 force-pushed the 450-bulkhead-docs-not-resolving branch from 5381b80 to 5eb8ce0 Compare February 20, 2025 20:48
@datho7561 datho7561 requested a review from rgrunber February 20, 2025 20:49
@datho7561
Copy link
Contributor Author

See #487

@datho7561 datho7561 closed this Feb 24, 2025
@datho7561 datho7561 removed the request for review from rgrunber February 24, 2025 14:54
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.

Tests failing
1 participant