I agree. I mean that the situation can be easily judged during the review process. So I think the automated check sames not so valuable. If you prefer, I have no objection.
Thanks ZhangJian He Enrico Olivelli <eolive...@gmail.com> 于2021年12月3日周五 17:22写道： > 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 > > > > > > > > > > > > > >
