-
Notifications
You must be signed in to change notification settings - Fork 3k
Dev MCP: Allow enable/disable of methods #50027
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
Dev MCP: Allow enable/disable of methods #50027
Conversation
81ab484
to
bc209e8
Compare
🙈 The PR is closed and the preview is expired. |
bc209e8
to
696e459
Compare
core/runtime/src/main/java/io/quarkus/runtime/annotations/DevMCPEnableAsDefault.java
Outdated
Show resolved
Hide resolved
696e459
to
9b4a6fc
Compare
Signed-off-by: Phillip Kruger <[email protected]>
9b4a6fc
to
42893d4
Compare
Signed-off-by: Phillip Kruger <[email protected]>
This comment has been minimized.
This comment has been minimized.
Status for workflow
|
Status for workflow
|
Status | Name | Step | Failures | Logs | Raw logs | Build scan |
---|---|---|---|---|---|---|
✔️ | JVM Tests - JDK 17 | Logs | Raw logs | 🚧 | ||
✔️ | JVM Tests - JDK 21 | Logs | Raw logs | 🚧 | ||
❌ | JVM Tests - JDK 17 Windows | Build |
Logs | Raw logs | 🚧 |
You can consult the Develocity build scans.
Flaky tests - Develocity
⚙️ JVM Tests - JDK 21
📦 extensions/smallrye-openapi/deployment
❌ io.quarkus.smallrye.openapi.test.vertx.OpenApiHttpRootPathCorsTestCase.testCorsFilterProperties
- History
1 expectation failed. Expected status code <200> but was <500>.
-java.lang.AssertionError
java.lang.AssertionError:
1 expectation failed.
Expected status code <200> but was <500>.
at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)
This PR adds further functionality to Dev MCP allowing extension developers to set a default enable/disable flag and for end users to toggle between these.
It also includes some updates to the extensions already supporting Dev MCP adding the flag to some of them where is make sense .
It also updates the documentation for Dev MCP to reflect this.
Fix #49903
// cc @myfear