Upgrade to latest struts
Struts devs have updated the 4.0.x struts2-bootstrap-plugin to work on the latest version of struts dependency> com.jgeppert.struts2.bootstrap struts2-bootstrap-plugin 4.0.2-SNAPSHOT It will be maintained also going forward (hopefully). Here is an upgraded version that I have done, only a few changes are needed. https://github.com/gregh3269/roller/tree/Upgrade_to_Struts-6.3.0.1 I have tested it and looks the same. Can you have a look? Need to get off the 2.5.x branch. Cheers Greg
[PR] Update JDK 21ea to 21ga in github actions. [roller]
mbien opened a new pull request, #132: URL: https://github.com/apache/roller/pull/132 this should fix the build -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@roller.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update JDK 21ea to 21ga in github actions. [roller]
mbien commented on PR #132: URL: https://github.com/apache/roller/pull/132#issuecomment-1750360937 all green -> merging -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@roller.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update JDK 21ea to 21ga in github actions. [roller]
mbien merged PR #132: URL: https://github.com/apache/roller/pull/132 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@roller.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: Upgrade to latest struts
Hi Greg, you could open a PR which will let tests run against it. This produces a dev build too for manual testing. on first glance: app/tools/roller.xml is probably not intended on the commit there are also a lot of whitespace changes in the commit. PR diffs can filter some of them out, but automated formatting changes in the javadoc for example should be avoided to make diffs smaller. Its probably an IDE setting somewhere. It is often better to separate cleanup and actual changes. -mbien On 06.10.23 11:51, Greg Huber wrote: Struts devs have updated the 4.0.x struts2-bootstrap-plugin to work on the latest version of struts dependency> com.jgeppert.struts2.bootstrap struts2-bootstrap-plugin 4.0.2-SNAPSHOT It will be maintained also going forward (hopefully). Here is an upgraded version that I have done, only a few changes are needed. https://github.com/gregh3269/roller/tree/Upgrade_to_Struts-6.3.0.1 I have tested it and looks the same. Can you have a look? Need to get off the 2.5.x branch. Cheers Greg
Re: Upgrade to latest struts
OK, a lot of white space, although I did not use format (eclipse) so not sure what happened. (Although there is some strange formatting going on that will never match any standard formatting, probably done manually). I will tidy it up, and try again. Cheers Greg On Fri, 6 Oct 2023 at 11:42, Michael Bien wrote: > Hi Greg, > > you could open a PR which will let tests run against it. This produces a > dev build too for manual testing. > > on first glance: > > app/tools/roller.xml is probably not intended on the commit > > there are also a lot of whitespace changes in the commit. PR diffs can > filter some of them out, but automated formatting changes in the javadoc > for example should be avoided to make diffs smaller. Its probably an IDE > setting somewhere. It is often better to separate cleanup and actual > changes. > > -mbien > > On 06.10.23 11:51, Greg Huber wrote: > > Struts devs have updated the 4.0.x struts2-bootstrap-plugin to work on > > the latest version of struts > > > > dependency> > > com.jgeppert.struts2.bootstrap > > struts2-bootstrap-plugin > > 4.0.2-SNAPSHOT > > > > > > It will be maintained also going forward (hopefully). > > > > > > Here is an upgraded version that I have done, only a few changes are > > needed. > > > > https://github.com/gregh3269/roller/tree/Upgrade_to_Struts-6.3.0.1 > > > > > > I have tested it and looks the same. Can you have a look? Need to > > get off the 2.5.x branch. > > > > > > Cheers Greg > > > > > >