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

LOC Seems not to be working for .scala files #7

Open
jamestidman opened this issue Dec 2, 2015 · 4 comments
Open

LOC Seems not to be working for .scala files #7

jamestidman opened this issue Dec 2, 2015 · 4 comments

Comments

@jamestidman
Copy link

LOC metrics seem not to be working. I have a Spray app and it will count the .properties file towards LOC but not the .scala files. The normal "lines" count seems fine though.

@hcoles
Copy link
Contributor

hcoles commented Dec 2, 2015

This plugin isn't a general scala plugin for sonar - it concentrates very narrowly on wrapping up scala style. It therefore doesn't provide a LOC count.

There did use to be an Scala plugin linked to from the sonarqube page that provided these basic metrics, but it looks to now have been deleted from github.

I think there needs to be a small plugin that provides very basic features such as this that could be used in conjunction with the scalastyle plugin and @RadoBuransky scoverage plugin.

@dthuering
Copy link

I see a similar behaviour.
There are even no warnings for my scala files while the scalastyle IntelliJ plugins has some.
Did I miss some settings?

@jonfreedman
Copy link

Would you consider accepting a patch to add this functionality?

@hcoles
Copy link
Contributor

hcoles commented Mar 5, 2016

@jonfreedman Yes we'd consider a patch.

Longer term I think we'd probably move the functionality out of this plugin to a small basic scala one as per my earlier comment, but it could start life here.

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

No branches or pull requests

4 participants