Il giorno ven 3 dic 2021 alle ore 10:20 ZhangJian He <shoot...@gmail.com> ha scritto:
> Gradle has `gradle-wrapper.jar` too. I think we don't need an automated > check, the reviewers can find if it's reasonable. > For some files there are specific acceptance rules. But we cannot commit other files that are not needed. Enrico > > Enrico Olivelli <eolive...@gmail.com> 于2021年11月10日周三 16:47写道： > > > ping > > > > > > Il giorno ven 5 nov 2021 alle ore 09:23 Enrico Olivelli < > > eolive...@gmail.com> > > ha scritto: > > > > > Hello, > > > This patch [1] contains a .jar file that must not be committed to the > > > source repository. > > > We generally cannot commit binary files to the source repo, as it won't > > be > > > "open source" anymore. > > > > > > When we are permitted to commit binary/compiled code, there must be a > > very > > > good reason and it must be approved explicitly and noted somewhere > > (NOTICE > > > file?) > > > > > > My point here is that we are missing some automated check to prevent > such > > > accidental commits. > > > I believe that we have to add a check to be run at every PR validation > > and > > > during the release process that our source distribution does not > contain > > > compiled code. > > > > > > Thoughts ? > > > > > > Enrico > > > > > > [1] https://github.com/apache/pulsar/pull/12535 > > > > > > > > >
