seems I will fix all the checkstyle related stuff later. This needs some serious cleanup.
We should move all the stuff in our root pom.xml into an own parent/pom.xml and add a checkstyle module. Then we can add the checkstyle rules as dependency and do not need to carry around all the stuff via absolute file paths. LieGrue, strub ----- Original Message ----- > From: Rick Curtis <[email protected]> > To: [email protected] > Cc: > Sent: Tuesday, 29 October 2013, 19:18 > Subject: Re: release preparations for 2.3.0 > > +1 on updating the checkstyle > > > > On Tue, Oct 29, 2013 at 12:54 PM, Mark Struberg <[email protected]> wrote: > >> I've already removed all unused imports in 2.3.x and trunk. >> Will remove all tabs and replace them with 4 characters tonight as well. >> >> We might also activate the maven-checkstyle-plugin to make sure this >> doesn't get worse in the future. >> >> LieGrue, >> strub >> >> >> >> >> ----- Original Message ----- >> > From: Mark Struberg <[email protected]> >> > To: openjpa-dev <[email protected]> >> > Cc: >> > Sent: Tuesday, 29 October 2013, 9:27 >> > Subject: release preparations for 2.3.0 >> > >> > Hi! >> > >> > Could you please walk through the unresolved OpenJPA-2.3.0 issues and >> set those >> > who you doubt should be fixed already to 'resolved'? >> > >> > >> > https://issues.apache.org/jira/issues/?jql=project%20%3D%20OPENJPA%20AND%20fixVersion%20%3D%20%222.3.0%22%20AND%20resolution%20%3D%20Unresolved%20ORDER%20BY%20due%20ASC%2C%20priority%20DESC%2C%20created%20ASC >> > >> > I hope to start with the actual release task either tonight or > tomorrow >> morning. >> > Means I gonna send another mail in 15 hours and then will transit the >> still >> > unresolved issues to 2.4.0. >> > >> > There is also another thing which I like to do: finally get rid of all >> the bad >> > tabs and unused imports in 2.3.x and trunk. >> > See OPENJPA-2200. I will _not_ change any lines other than tabs and >> unused >> > imports so far. >> > Patches should thus still apply fine even across older versions. >> > >> > LieGrue, >> > strub >> > >> > > > > -- > *Rick Curtis* >
