Re: Missing check on .jar files committed to the source repo

2021-12-03 Thread Dave Fisher
> On Dec 3, 2021, at 11:45 AM, Michael Marshall wrote: > >> Automated checks are useful because we are human and we usually miss to >> validate this kind of boring stuff. > > +1 I think it sounds appropriate to add an automated check. If a use > case arises where we need to add compiled files

Re: Missing check on .jar files committed to the source repo

2021-12-03 Thread Michael Marshall
> Automated checks are useful because we are human and we usually miss to > validate this kind of boring stuff. +1 I think it sounds appropriate to add an automated check. If a use case arises where we need to add compiled files, we'll also need a way to bypass/override this check. Michael On Fr

Re: Missing check on .jar files committed to the source repo

2021-12-03 Thread Enrico Olivelli
Il giorno ven 3 dic 2021 alle ore 10:36 ZhangJian He ha scritto: > 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. > Automated checks are useful because we are human a

Re: Missing check on .jar files committed to the source repo

2021-12-03 Thread ZhangJian He
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 于2021年12月3日周五 17:22写道: > Il giorno ven 3 dic 2021 alle ore 10:20 ZhangJian He > ha s

Re: Missing check on .jar files committed to the source repo

2021-12-03 Thread Enrico Olivelli
Il giorno ven 3 dic 2021 alle ore 10:20 ZhangJian He 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 neede

Re: Missing check on .jar files committed to the source repo

2021-12-03 Thread ZhangJian He
Gradle has `gradle-wrapper.jar` too. I think we don't need an automated check, the reviewers can find if it's reasonable. Enrico Olivelli 于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 patc

Re: Missing check on .jar files committed to the source repo

2021-11-10 Thread Enrico Olivelli
ping Il giorno ven 5 nov 2021 alle ore 09:23 Enrico Olivelli 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 pe

Missing check on .jar files committed to the source repo

2021-11-05 Thread Enrico Olivelli
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 ap