Yes, that's definitely not a problem. I was simply stating that merging the branch is not sufficient and that the real integration needs to be done.
Le mer. 9 oct. 2024 à 09:21, Xeno Amess <xenoam...@gmail.com> a écrit : > RAT checks has a excluding rule, if we really wanna so we can just exclude > that folder anyway... > > Guillaume Nodet <gno...@apache.org> 于2024年10月9日周三 15:11写道: > > > I just pushed a branch with the results: > > https://github.com/gnodet/maven/tree/merge-its > > > > This is just the raw output of the recipe. The ITs are not integrated > into > > the build, which will even fail due to RAT checks and maybe other > reasons. > > > > Le mer. 9 oct. 2024 à 08:35, Herve Boutemy <hbout...@apache.org> a > écrit : > > > > > I understand how it adds complexity > > > > > > AFAIK, the interest of having a separate project of core ITs is to > clear > > > state the Maven core version range for each test, to clearly document > > when > > > things were introduced / broken / fixed and even be able to run HEAD > ITs > > > against a past release > > > > > > is it the only reason? I don't know, it's the key aspect I understood > 15 > > > years ago when it was done and I was too noob to really grasp every > > detail > > > :) > > > > > > does this really deserve the complexity it creates? > > > I don't know > > > > > > I also need to check the Git merge recipe, to see how the result would > be > > > ok to me: do you have a personal fork somewhere so we can review > without > > > running the command ourselves? > > > > > > Regards, > > > > > > Hervé > > > > > > On 2024/10/08 06:36:19 Guillaume Nodet wrote: > > > > I'd like to discuss merging ITs into maven core repository. > > > > The ITs have already been splitted some time ago between the 3.x > branch > > > and > > > > master for testing Maven master. But I don't really see a good > reason > > to > > > > keep the repositories separated, this makes things more complicated > > when > > > > modifying maven code and adding ITs. > > > > > > > > The following script allow merging the two repositories while keeping > > > both > > > > histories: > > > > > > > > > > > > brew install git-filter-repo > > > > > > > > git clone https://github.com/apache/maven > > > > > > > > git clone https://github.com/apache/maven-integration-testing > > > > > > > > (cd maven-integration-testing && \ > > > > > > > > git filter-repo --to-subdirectory-filter its) > > > > > > > > (cd maven && \ > > > > > > > > git remote add its ../maven-integration-testing && \ > > > > > > > > git fetch its --no-tags && \ > > > > > > > > EDITOR=true git merge --allow-unrelated-histories its/master && \ > > > > > > > > git remote remove its) > > > > > > > > The next step would be to actually include them in a profile and > update > > > the > > > > github workflow. > > > > I think they could be refactored to: > > > > * first perform a full run on Ubuntu + latest LTS JDK: > > > > - restore cache > > > > - checkout > > > > - build with no tests > > > > - run IT bootstrap (to prime local repo) > > > > - save cache > > > > - build again with tests and ITs > > > > * if this first run succeeds, do the same on other platforms / jdks > > > > > > > > The cache is important to add imho. It seems lately, GH runners have > > > often > > > > problems downloading from maven central, so that would help a lot > > > > increasing the stability. So using > > > > https://github.com/marketplace/actions/maven-cache or a similar > action > > > > would be handy imho. > > > > > > > > We should also upload nightlies from GH. > > > > And automate the releases as much as possible.... > > > > > > > > -- > > > > ------------------------ > > > > Guillaume Nodet > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > > > For additional commands, e-mail: dev-h...@maven.apache.org > > > > > > > > > > -- > > ------------------------ > > Guillaume Nodet > > > -- ------------------------ Guillaume Nodet