-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Disable graphql-client quickstart tests for now #1486
Disable graphql-client quickstart tests for now #1486
Conversation
Wouldn't just disabling the quickstart's test be a less brutal solution? :) |
Ok, I can do that :) |
6968560
to
eaef7a1
Compare
@jmartisk updated |
eaef7a1
to
d846cea
Compare
I needed to put |
We don't need this PR right as you found an alternative? |
Not sure what you mean, we haven't implemented any alternative approach yet |
Alternative was possible for |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
d846cea
to
971fa44
Compare
Rebased to get #1484 fix for failing vertx-quickstart |
Status for workflow
|
@jmartisk all green now ;) |
Disable graphql-client quickstart tests for now
Blocked by quarkusio/quarkus#45334
Check list:
Your pull request:
development
branch999-SNAPSHOT
version of Quarkusmvn clean test
)mvn clean package -Pnative
)mvn clean verify -Pnative
)README.md
file (with build and run instructions)pom.xml
andREADME.md