Re: Eclipse formatting, was: Martin's wicket pull request

2013-11-08 Thread Martin Grigorov
I'll test this soon. I'll update the docs for IDEA too if needed. On Thu, Nov 7, 2013 at 11:02 AM, Sven Meier s...@meiers.net wrote: Thanks, I've added a hint to the Idea instructions. Regards Sven On 11/06/2013 10:12 AM, Vojtěch Krása wrote: You should also specify values for Class

WICKET-4997

2013-11-08 Thread Emond Papegaaij
Hi, 2 weeks ago, we decided to merge WICKET-4997 into master first, and into wicket-6.x after the 6.12 release. As 6.12 was released this week, I would like to go forward and merge the branch into 6.x. Any objections? Best regards, Emond

remove recently merged cdi 1.1 from 6.x

2013-11-08 Thread Igor Vaynberg
not sure why this was merged into 6.x but there are a lot of problems with this contribution as can be seen here [1]. i think this should be removed from at least the release branch. -igor [1] https://github.com/apache/wicket/pull/50#issuecomment-28063112

Re: remove recently merged cdi 1.1 from 6.x

2013-11-08 Thread John Sarman
+1 removal Never should have been merged into the 6 branch and not the 7 until there is a consensus. On Fri, Nov 8, 2013 at 4:07 PM, Igor Vaynberg igor.vaynb...@gmail.comwrote: not sure why this was merged into 6.x but there are a lot of problems with this contribution as can be seen here

Re: Eclipse formatting, was: Martin's wicket pull request

2013-11-08 Thread Igor Vaynberg
it is really frustrating that i have to do this manually now. before all i had to do was checkout the project and it was all set. wicket shares my workspace with other projects so the workspace-default is not going to work. can we drop the format def on wicket.apache.org and configure the maven