Thanks a lot for bringing this to the mailing list. I'm also in favor of us going with the Apache RAT CLI tool, with one .rat-excludes per repository.
On Thu, Nov 14, 2024 at 12:20 PM Jan Šťastný <[email protected]> wrote: > > +1 for CLI > I'd use a one per repository, which then ends up in the source zip archive > and is straightforward for all voters to execute the same file in several > directories. Also allows to implement a PR check like this, or similar. > > regards > Jan > > On Thu, 14 Nov 2024 at 16:26, Jason Porter <[email protected]> wrote: > > > Do we have a ticket for this? > > > > Also, do we have a single .rat-excludes at the root of the release > > tarball, or one for each repo? Maybe during the release we combine all of > > the ones from the different repos into one so it is easier for people to > > use during release time? > > > > -- > > Jason Porter > > Software Engineer > > He/Him/His > > > > IBM > > > > > > From: Jozef Marko <[email protected]> > > Date: Thursday, November 14, 2024 at 02:27 > > To: [email protected] <[email protected]> > > Subject: [EXTERNAL] Re: [DISCUSS] license headers check > > Hi Toshiya, > > > > thank you for this email. Yes, just sharing also here my opinion, that > > 'apache-rat java cli tool' fits the best the current state of the KIE > > codebase, where we have also the kie-tools monorepo, that is not bound to > > the maven. > > > > > > Jozef Marko > > > > Software Developer > > > > [email protected] > > > > > > > > ________________________________ > > From: Toshiya Kobayashi <[email protected]> > > Sent: Thursday, November 14, 2024 7:25 AM > > To: [email protected] <[email protected]> > > Subject: [EXTERNAL] [DISCUSS] license headers check > > > > Hi, > > > > One of the cumbersome parts of the release verification is "All files have > > license headers if necessary." > > > > If we have a .rat-excludes file in each repository, the step can be quickly > > done with the command line. > > > > I filed a GH issue: > > https://github.com/apache/incubator-kie-issues/issues/1613 > > > > Jozef mentioned that we should unify the approach (Thanks!), so I raise the > > topic in this ML. > > > > Please have a look at Jozef's comment about the existing initiatives for > > license check. > > > > > > https://github.com/apache/incubator-kie-issues/issues/1613#issuecomment-2475467000 > > > > As written in the comment, we should choose one of them. > > > > - apache-rat-plugin > > - checkstyle.header.template > > - apache-rat java cli tool > > > > Personally, +1 for `apache-rat java cli tool` because it's not tied to > > maven projects and it's easy for the release verification step. > > > > Please share any thoughts on this. Thanks! > > > > Toshiya > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
