jira.codehaus.org

  • Log In Access more options
    • Online Help
    • Keyboard Shortcuts
    • About JIRA
    • JIRA Credits
    • What?s New
  • Dashboards Access more options (Alt+d)
  • Projects Access more options (Alt+p)
  • Issues Access more options (Alt+i)
Signup
Sonar Plugins
  • Sonar Plugins
  • SONARPLUGINS-956

Code best practices

  • Log In
  • Views
    • XML
    • Word
    • Printable

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: JMETER-0.1
  • Fix Version/s: JMETER-0.1
  • Component/s: JMeter
  • Labels:
    None
  • Number of attachments :
    0

Description

Before vote will occur :

  • Could you please re-read http://docs.codehaus.org/display/SONAR/Developer+guide#Developerguide-Codeformatting - replace tabs by two spaces.
  • The expression $ {pom.groupId}

    is deprecated. Please use $

    {project.groupId}

    instead.

  • It's not recommended to have "repositories" section in pom.xml, so I wonder if it really required?
  • Configuration of sonar-packaging-maven-plugin should be removed from sonar-jmeter-plugin. Use property "sonar.pluginClass". And don't set plugin key - it would be generated automatically from artifactId. See http://docs.codehaus.org/display/SONAR/sonar-packaging-maven-plugin
  • Configuration for maven-compiler-plugin and "project.build.sourceEncoding" should be removed from POMs - see http://docs.codehaus.org/display/SONAR/Sonar+Plugins+Forge+Parent+POM
  • Common stuff (like organization, license, developers) should be declared in parent ( org.codehaus.sonar-plugins.jmeter:parent ) and you can remove version from sub-projects, since anyway they should be released together.
  • Use package "org.sobar.plugins.jmeter".

Activity

  • All
  • Comments
  • Work Log
  • History
  • Activity
Hide
Permalink
Antonio Calero added a comment - 03/Feb/11 4:39 PM

Done

Show
Antonio Calero added a comment - 03/Feb/11 4:39 PM Done

People

  • Assignee:
    Antonio Calero
    Reporter:
    Antonio Calero
Vote (0)
Watch (0)

Dates

  • Created:
    03/Feb/11 9:41 AM
    Updated:
    03/Feb/11 4:39 PM
    Resolved:
    03/Feb/11 4:39 PM
  • Atlassian JIRA (v5.2.7#850-sha1:b2af0c8)
  • Report a problem
  • Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.