Skip to content

Conversation

@matejvasek
Copy link
Contributor

No description provided.

Signed-off-by: Matej Vasek <mvasek@redhat.com>
@slinkydeveloper
Copy link
Member

Did you tried to run the maven javadoc plugin locally? since you're here, can you enable it for the whole build so we make sure we catch those issues next time before merging?

@matejvasek
Copy link
Contributor Author

I did run javadoc build.

@matejvasek
Copy link
Contributor Author

@slinkydeveloper we could move javadoc plugin up

diff --git a/pom.xml b/pom.xml
index 1185628..30b7d4b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -142,6 +142,22 @@
                 </plugin>
             </plugins>
         </pluginManagement>
+        <plugins>
+            <!-- Javadoc jar -->
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-javadoc-plugin</artifactId>
+                <version>2.10.4</version>
+                <executions>
+                    <execution>
+                        <id>attach-javadocs</id>
+                        <goals>
+                            <goal>jar</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
+        </plugins>
     </build>
 
     <profiles>
@@ -198,22 +214,6 @@
                             </execution>
                         </executions>
                     </plugin>
-
-                    <!-- Javadoc jar -->
-                    <plugin>
-                        <groupId>org.apache.maven.plugins</groupId>
-                        <artifactId>maven-javadoc-plugin</artifactId>
-                        <version>2.10.4</version>
-                        <executions>
-                            <execution>
-                                <id>attach-javadocs</id>
-                                <goals>
-                                    <goal>jar</goal>
-                                </goals>
-                            </execution>
-                        </executions>
-                    </plugin>
-
                     <plugin>
                         <groupId>org.apache.maven.plugins</groupId>
                         <artifactId>maven-gpg-plugin</artifactId>

@slinkydeveloper
Copy link
Member

What do you mean? Can you commit this change?

@matejvasek
Copy link
Contributor Author

#275

@slinkydeveloper
Copy link
Member

Cool, I'm gonna merge this one meanwhile

@slinkydeveloper slinkydeveloper merged commit 62fe155 into cloudevents:master Nov 13, 2020
@slinkydeveloper slinkydeveloper added this to the 2.0.0.CR1 milestone Nov 13, 2020
@slinkydeveloper slinkydeveloper added bug Something isn't working documentation labels Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants