-
Notifications
You must be signed in to change notification settings - Fork 465
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
[WFCORE-6750][Preview] Integrate the scanner for the unstable API annotations #5913
Conversation
Dependency Tree Analyzer Output: New Dependencies:
CC @wildfly/prod |
...main/java/org/wildfly/extension/core/management/UnstableApiAnnotationResourceDefinition.java
Show resolved
Hide resolved
...m/src/main/java/org/wildfly/extension/core/management/CoreManagementSubsystemSchema_1_0.java
Outdated
Show resolved
Hide resolved
fe1cd49
to
d26d428
Compare
d26d428
to
5d6eb7a
Compare
@@ -16,6 +16,11 @@ core-management.configuration-changes.remove=Remove the configuration changes an | |||
core-management.configuration-changes.max-history=The maximum number of configuration changes stored in history. | |||
core-management.configuration-changes.list-changes=List the last configuration changes. | |||
|
|||
core-management.unstable-api-annotations=Service to configure how we deal with finding annotations indicating unstable api in user code. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/api/API
core-management.unstable-api-annotations=Service to configure how we deal with finding annotations indicating unstable api in user code. | ||
core-management.unstable-api-annotations.add=Add the history for configuration changes. | ||
core-management.unstable-api-annotations.remove=Remove the configuration changes and clear the history. | ||
core-management.unstable-api-annotations.level=Whether to throw an error or log a warning if annotations indicating unstable api. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/api/API
/* | ||
* JBoss, Home of Professional Open Source. | ||
* Copyright 2024 Red Hat, Inc., and individual contributors | ||
* as indicated by the @author tags. | ||
* | ||
* Licensed under the Apache License, Version 2.0 (the "License"); | ||
* you may not use this file except in compliance with the License. | ||
* You may obtain a copy of the License at | ||
* | ||
* http://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* Unless required by applicable law or agreed to in writing, software | ||
* distributed under the License is distributed on an "AS IS" BASIS, | ||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
* See the License for the specific language governing permissions and | ||
* limitations under the License. | ||
* | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is really new code we want to use the updated license header in Java files:
/*
* Copyright The WildFly Authors
* SPDX-License-Identifier: Apache-2.0
*/
5d6eb7a
to
7d56a1e
Compare
Core -> Full Integration Build 13598 outcome was FAILURE using a merge of 7d56a1e |
Core -> WildFly Preview Integration Build 13660 outcome was FAILURE using a merge of 7d56a1e |
Core -> Full Integration Build 13864 outcome was FAILURE using a merge of 7d56a1e |
7d56a1e
to
f096b56
Compare
Core -> Full Integration Build 13599 outcome was FAILURE using a merge of f096b56 Failed tests
|
Core -> WildFly Preview Integration Build 13661 outcome was FAILURE using a merge of f096b56 Failed tests
|
f096b56
to
6717755
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a small provisioning question
<module name="org.wildfly.extension.core-management-client"/> | ||
<module name="org.wildfly.security.elytron-private"/> | ||
<module name="org.wildfly.service"/> | ||
<module name="org.wildfly.unstable.annotation.api.indexer"/> | ||
<module name="org.wildfly._internal.unstable-api-annotation-index"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't those modules be optional
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ehsavoie The indices and the dependency on the scanner need to always be added, or we would need a layer which doesn't seem practical for a preview feature. A bit further down the line (once this reaches community level) I would like to add a layer which enables the configuration to turn the scanning on. But still there is a hard dependency on the indexer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I was wondering about adding all those indexes and jars if one just doesn't want the feature
7941c34
to
718d766
Compare
Core -> WildFly Preview Integration Build 13685 outcome was FAILURE using a merge of 718d766 Failed tests
|
Core -> Full Integration Build 13889 outcome was FAILURE using a merge of 718d766 Failed tests
|
...main/java/org/wildfly/extension/core/management/UnstableApiAnnotationResourceDefinition.java
Show resolved
Hide resolved
...rc/main/java/org/wildfly/extension/core/management/CoreManagementRootResourceDefinition.java
Outdated
Show resolved
Hide resolved
...m/src/main/java/org/wildfly/extension/core/management/CoreManagementSubsystemSchema_1_0.java
Outdated
Show resolved
Hide resolved
...m/src/main/java/org/wildfly/extension/core/management/CoreManagementSubsystemSchema_1_0.java
Outdated
Show resolved
Hide resolved
...rc/test/java/org/wildfly/extension/core/management/CoreManagementSubsystem_1_0_TestCase.java
Outdated
Show resolved
Hide resolved
...system/src/main/java/org/wildfly/extension/core/management/UnstableApiAnnotationService.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/wildfly/extension/core/management/CoreManagementRootResourceDefinition.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/wildfly/extension/core/management/CoreManagementRootResourceDefinition.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/wildfly/extension/core/management/CoreManagementRootResourceDefinition.java
Outdated
Show resolved
Hide resolved
718d766
to
7453eb3
Compare
...main/java/org/wildfly/extension/core/management/UnstableApiAnnotationResourceDefinition.java
Outdated
Show resolved
Hide resolved
.../java/org/wildfly/extension/core/management/deployment/UnstableApiAnnotationAttachments.java
Outdated
Show resolved
Hide resolved
7453eb3
to
77c2b08
Compare
https://issues.redhat.com/browse/WFCORE-6750
Proposal: wildfly/wildfly-proposals#569
The library lives in https://github.com/wildfly/unstable-api-annotation-utils. Once approved I will release a 1.0.0.Final.
The full PR is wildfly/wildfly#17743 and needs this one merged before it can pass CI