Fix the skip-copyright profile #310
Conversation
|
Dit gaat zo niet goed. Als je met deze code het skip-copyright niet enabled, dan heb je toch niet de copyright check. Ik ga proberen deze PR uit te breiden zodat je de copyright check nog steeds kan doen als je dat wilt. |
| <plugin> | ||
| <groupId>org.glassfish.copyright</groupId> | ||
| <artifactId>glassfish-copyright-maven-plugin</artifactId> | ||
| <version>2.4</version> | ||
| <executions> | ||
| <execution> | ||
| <id>Check-CopyRight-Notice</id> | ||
| <goals> | ||
| <goal>copyright</goal><!-- Checks for Copyright notices --> | ||
| <goal>check</goal><!-- Fails the build --> | ||
| </goals> | ||
| <phase>process-sources</phase> | ||
| </execution> | ||
| </executions> | ||
| <configuration> | ||
| <excludeFile>${ibis.ladybug.rootdir}${file.separator}copyrightExcludeFile.txt</excludeFile> | ||
| <useDash>true</useDash> | ||
| <templateFile>${ibis.ladybug.rootdir}/copyrightTemplateFile.txt</templateFile> | ||
| </configuration> | ||
| </plugin> |
There was a problem hiding this comment.
Stond dit eerst ergens anders gedefineerd?
There was a problem hiding this comment.
Ik denk onder het skip-copyright profile, dat per abous juist wél de copyright check doet. Dat zie je aan de linkerkant van de vergelijking: <phase>process-sources</phase>.
|
Sergi heeft even meegekeken. Het werkte al zoals het was, maar je moet het profiel via een property runnen. Het profiel heeft een misleidende naam, want juist binnen de definitie van het profiel staat dat de copyright check wél gebeurt. Ik zal een nieuwe PR maken met een commentaar dat hierop wijst. |
No description provided.