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

Add SQALE model definitions for the built-in checks #199

Closed
Bertk opened this issue May 4, 2014 · 8 comments
Closed

Add SQALE model definitions for the built-in checks #199

Bertk opened this issue May 4, 2014 · 8 comments
Assignees
Milestone

Comments

@Bertk
Copy link
Contributor

Bertk commented May 4, 2014

The current definition for Sqale model does not specify remediation costs for cxx-checks like FunctionCyclomaticComplexity, FileCyclomaticComplexity, ... and manual updates of cxx-model.xml is a major effort.

Therefore all available cxx-checks shall be added to the Sqale model and also the cxx-model.xml file shall be formatted in a structured way to allow authoring and merging of customized settings.

@jmecosta
Copy link
Member

jmecosta commented May 4, 2014

I think as of sonar 4.3 it should be possible to define remediation costs
when check is defined via annotations in the code. This solves the cxx
checks problem.

Regarding the model, might be a good idea just to keep the cxx-model.XML
the other was ok to create the initial version, now the effort to update it
might be smaller and should be managable via pull requests?

On May 4, 2014 12:54 PM, "Bert" notifications@github.com wrote:

The current definition for Sqale model does not specify remediation costs
for cxx-checks like FunctionCyclomaticComplexity, FileCyclomaticComplexity,
... and manual updates of cxx-model.xml is a major effort.

Therefore all available cxx-checks shall be added to the Sqale model and
also the cxx-model.xml file shall be formatted in a structured way to allow
authoring and merging of customized settings.


Reply to this email directly or view it on GitHub.

@Bertk
Copy link
Contributor Author

Bertk commented May 4, 2014

Defining techn. debt using annotations will of course solves this issue for the future.
For me it will be easier to merge pull requests or even extent the definitions in cxx-model.xml if it is somehow sorted - see solution for merging vcproj files with VcprojFormatter.exe (http://www.codeproject.com/Articles/133604/Visual-C-version-vcproj-project-file-formatter). Unfortunately this kind of tool is not available for sqale models.

@jmecosta
Copy link
Member

jmecosta commented May 4, 2014

is it ok for you if i write a simple application that takes uses only the
cxx-model. This application can have a simple data grid, and a few buttons
to add and remove characteristics?

On Sun, May 4, 2014 at 2:19 PM, Bert notifications@github.com wrote:

Defining techn. debt using annotations will of course solves this issue
for the future.
For me it will be easier to merge pull requests or even extent the
definitions in cxx-model.xml if it is somehow sorted - see solution for
merging vcproj files with VcprojFormatter.exe (
http://www.codeproject.com/Articles/133604/Visual-C-version-vcproj-project-file-formatter).
Unfortunately this kind of tool is not available for sqale models.


Reply to this email directly or view it on GitHubhttps://github.com//issues/199#issuecomment-42130538
.

@Bertk
Copy link
Contributor Author

Bertk commented May 4, 2014

That is OK for me. So I do not need to create a XSL template to sort cxx-model.xml.

@wenns wenns added this to the M 0.9.2 milestone Jul 2, 2014
@wenns
Copy link
Contributor

wenns commented Dec 18, 2014

Hi guys! is anyone interested in making a pull request for this? The annotations Jorge spoke about seem to be still not in use, so this would involve editing cxx-model.xml

@jmecosta
Copy link
Member

The best for now is to add those characteristics directly I to model I can
try to provide those

On Thu, Dec 18, 2014, 12:37 Waleri Enns notifications@github.com wrote:

Hi guys! is anyone interested in making a pull request for this? The
annotations Jorge spoke about seem to be still not in use, so this would
involve editing cxx-model.xml


Reply to this email directly or view it on GitHub
#199 (comment).

@wenns wenns changed the title add techn. debt for cxx-checks to sqale model Add SQALE model definitions for the built-in checks Dec 18, 2014
@jmecosta
Copy link
Member

#330

@wenns
Copy link
Contributor

wenns commented Dec 22, 2014

Closed by merging #330

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants