Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cleanup pom.xml #361

Closed
stdweird opened this issue Oct 30, 2014 · 4 comments
Closed

cleanup pom.xml #361

stdweird opened this issue Oct 30, 2014 · 4 comments
Milestone

Comments

@stdweird
Copy link
Member

We are in need of uniform pom.xml in all components.

@stdweird stdweird added this to the 14.12 milestone Oct 30, 2014
@stdweird
Copy link
Member Author

also in need of some style guide wrt readability and how to remove the dummy entries in develpers/contributors.

@jouvin
Copy link
Contributor

jouvin commented Oct 30, 2014

I may try to have a look after the 14.10 release. I did quite some cleanups in the existing pom.xml files (at least some of them). As for the developers/contributors, the idea was to keep the pom.xml unmodified and enhance the plugin to process both the same way... See quattor/maven-tools#16.

@stdweird
Copy link
Member Author

wrt the contributors, i was taking about the Jane SMITH / Joe DOE entries that make no sense whatsoever.
wrt cleanup, quite some components have

  <build>
    <pluginManagement>
      <plugins>
        <plugin>
          <groupId>org.quattor.pan</groupId>
          <artifactId>panc-maven-plugin</artifactId>
          <version>9.0.0-RC3</version>
        </plugin>
      </plugins>
    </pluginManagement>
  </build>

which is not required at all. (i don't even know what it is supposed to do)

@jouvin
Copy link
Contributor

jouvin commented Oct 30, 2014

Probably it can be removed as it is in build-profile... See quattor/release#53. I suggest that we keep all these cleanups for the beginning of the next development cycle...

jouvin added a commit to jouvin/configuration-modules-core that referenced this issue Dec 2, 2014
jouvin added a commit to jouvin/configuration-modules-core that referenced this issue Dec 9, 2014
@jouvin jouvin closed this as completed in 03604ad Dec 15, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants