You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tamaya.apache.org by an...@apache.org on 2018/11/18 21:21:18 UTC

[14/22] incubator-tamaya git commit: TAMAYA-274/353 Added revapi reports to control/report backward compatibility breaks. Removed deprecated artifacts (ConfigurationContextBuilder) and implementations/deps. TAMAYA-355 Removed PropertyValueC

http://git-wip-us.apache.org/repos/asf/incubator-tamaya/blob/606f8ec6/code/spi-support/src/test/java/org/apache/tamaya/spisupport/TestConfigurationProvider.java
----------------------------------------------------------------------
diff --git a/code/spi-support/src/test/java/org/apache/tamaya/spisupport/TestConfigurationProvider.java b/code/spi-support/src/test/java/org/apache/tamaya/spisupport/TestConfigurationProvider.java
index b20e02a..cf60849 100644
--- a/code/spi-support/src/test/java/org/apache/tamaya/spisupport/TestConfigurationProvider.java
+++ b/code/spi-support/src/test/java/org/apache/tamaya/spisupport/TestConfigurationProvider.java
@@ -21,7 +21,6 @@ package org.apache.tamaya.spisupport;
 import org.apache.tamaya.Configuration;
 import org.apache.tamaya.spi.ConfigurationBuilder;
 import org.apache.tamaya.spi.ConfigurationContext;
-import org.apache.tamaya.spi.ConfigurationContextBuilder;
 import org.apache.tamaya.spi.ConfigurationProviderSpi;
 import org.osgi.service.component.annotations.Component;
 
@@ -56,11 +55,6 @@ public class TestConfigurationProvider implements ConfigurationProviderSpi {
     }
 
     @Override
-    public ConfigurationContextBuilder getConfigurationContextBuilder() {
-        return new DefaultConfigurationContextBuilder();
-    }
-
-    @Override
     public void setConfiguration(Configuration config, ClassLoader classLoader) {
         Objects.requireNonNull(config.getContext());
         this.config = Objects.requireNonNull(config);
@@ -71,22 +65,4 @@ public class TestConfigurationProvider implements ConfigurationProviderSpi {
         return true;
     }
 
-    /**
-     * @deprecated use {@link Configuration#getContext()} instead.
-     */
-    @Deprecated
-    @Override
-    public ConfigurationContext getConfigurationContext() {
-        return this.config.getContext();
-    }
-
-    /**
-     * @deprecated the context should be given upon creation of the {@link Configuration}
-     */
-    @Deprecated
-    @Override
-    public void setConfigurationContext(ConfigurationContext context){
-        this.config = new DefaultConfigurationBuilder(context).build();
-    }
-
 }

http://git-wip-us.apache.org/repos/asf/incubator-tamaya/blob/606f8ec6/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 716b48b..4867d54 100644
--- a/pom.xml
+++ b/pom.xml
@@ -72,6 +72,9 @@
         <reflow-skin.version>1.1.1</reflow-skin.version>
         <released_version>0.3-incubating</released_version>
         <osgi.annotation.version>6.0.0</osgi.annotation.version>
+        <!-- API checker -->
+        <revapi-java.version>0.18.0</revapi-java.version>
+        <revapi.plugin.version>0.10.5</revapi.plugin.version>
     </properties>
 
     <licenses>
@@ -293,6 +296,44 @@
         <pluginManagement>
             <plugins>
                 <plugin>
+                    <groupId>org.revapi</groupId>
+                    <artifactId>revapi-maven-plugin</artifactId>
+                    <version>${revapi.plugin.version}</version>
+                    <configuration>
+                        <failSeverity>breaking</failSeverity>
+                        <failBuildOnProblemsFound>false</failBuildOnProblemsFound>
+                        <analysisConfiguration>
+                            <revapi.reporter.text id="stdout">
+                                <minSeverity>POTENTIALLY_BREAKING</minSeverity>
+                                <output>out</output>
+                                <append>false</append>
+                            </revapi.reporter.text>
+                            <revapi.filter>
+                                <elements>
+                                    <exclude>
+                                        <item>.*internal.*</item>
+                                        <item>.*osgi.*</item>
+                                    </exclude>
+                                </elements>
+                            </revapi.filter>
+                        </analysisConfiguration>
+                    </configuration>
+                    <dependencies>
+                        <dependency>
+                            <groupId>org.revapi</groupId>
+                            <artifactId>revapi-java</artifactId>
+                            <version>${revapi-java.version}</version>
+                        </dependency>
+                    </dependencies>
+                    <executions>
+                        <execution>
+                            <id>report</id>
+                            <phase>package</phase>
+                            <goals><goal>report</goal></goals>
+                        </execution>
+                    </executions>
+                </plugin>
+                <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-assembly-plugin</artifactId>
                     <version>${assembly.version}</version>