I see 7083 is merged already, and the changes would be limited to conflicts only if somebody doesn't follow the new convention on having the file ending with newline or not.
On 7131, I've added some comments. I'm more concerned about its risks and rewards, compared to the merge conflicts it may cause. If we're doing the log4j upgrade, should this also consider other related dependencies such as gson? Thanks and regards. ________________________________ From: Wei ZHOU <ustcweiz...@gmail.com> Sent: Monday, March 27, 2023 13:15 To: dev@cloudstack.apache.org <dev@cloudstack.apache.org> Subject: Re: coming conflicts this might cause a lot of work when merge 4.17/4.18 into main branch as well On Mon, 27 Mar 2023 at 08:51, Daan Hoogland <daan.hoogl...@gmail.com> wrote: > Devs, > There are two PRs [1][2] to be merged soon that have several hundreds of > files involved. One is to unify the line-endings and the other one to > (finaly) upgrade log4j. please beware that this is going to cause you to > resolve conflicts in your PRs. > > regards, > > [1] https://github.com/apache/cloudstack/pull/7083 > [2] https://github.com/apache/cloudstack/pull/7131 > > -- > Daan >
