Re: [GitHub] maven-enforcer pull request: [MENFORCER-193]: Add new rule: Banned...

2014-06-04 Thread Wang, Simon
Any update for this PR? Please let me know if have any concerns. Regards Simon On Jun 2, 2014, at 10:16 PM, Wang, Simon wrote: > Thanks for your comments, Karl! > > I added document for this rule, please review it. > https://github.com/apache/maven-enforcer/pull/13 > >

Re: [GitHub] maven-enforcer pull request: [MENFORCER-193]: Add new rule: Banned...

2014-06-02 Thread Wang, Simon
y to quote the answers of the others here on the dev > list to posts cause obviously I'm subscribed on the dev list and I'm > following the discussion. > > On 5/29/14 1:52 PM, Wang, Simon wrote: >> Hi, Karl, >> >> Thanks for your comments. >> >> I

Re: enforcer-rules: standard vs. extra-enforcer-rules

2014-05-29 Thread Wang, Simon
Hi, Robert, Karl asked same question, please refer below mail about this question. Hope that help. Regards Simon Hi, Karl, Thanks for your comments. I did dig into requireNoRepositories.html, the purpose for that rule is: detect whether pom and pom’s parents contains repositories definitio

Re: enforcer-rules: standard vs. extra-enforcer-rules

2014-05-29 Thread Wang, Simon
Thanks for your comments, Mirko! This feature would be helpful for those enterprise maven developers. That would be great could include it into standard rules and benefit more enterprise users. Regards Simon On May 29, 2014, at 4:19 AM, Mirko Friedenhagen wrote: > Hello everybody, > > there

Re: [GitHub] maven-enforcer pull request: [MENFORCER-193]: Add new rule: Banned...

2014-05-29 Thread Wang, Simon
Hi, Karl, Thanks for your comments. I did dig into requireNoRepositories.html, the purpose for that rule is: detect whether pom and pom’s parents contains repositories definition. That make sense to guide users to use correct convention (not define repositories in pom files). But “BannedReposit

http://jira.codehaus.org/browse/MPIR-295

2014-05-22 Thread Wang, Simon
Hi, Team, Anyone could give a review for this enhancement? https://github.com/apache/maven-plugins/pull/23 It’s trying to make dependency convergence report to be more valuable. Regards Simon