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

Option to export found issues using Checkstyle #46

Open
enen92 opened this issue Mar 1, 2018 · 5 comments
Open

Option to export found issues using Checkstyle #46

enen92 opened this issue Mar 1, 2018 · 5 comments

Comments

@enen92
Copy link
Member

enen92 commented Mar 1, 2018

Checkstyle is an XML output format for source-code checkers. This will provide interoperability with both IDEs and editors that expect code suggestions in checkstyle format, as well as other continuous code quality platforms (e.g. personal instances of sonarqube). Checkstyle output format is built into many linters found out there (see for instance gometalinter: https://github.com/alecthomas/gometalinter#checkstyle-xml-format).

Exposing found issues in Kodi addons using a common standard will certainly help the development of plugins for most of existing IDEs.

This is not a high priority requirement, just food for though. At least keep the idea in mind while developing the tool further so it will not require drastic architectural changes if someone wants to implement this.
Cheers

@razzeee
Copy link
Member

razzeee commented Mar 5, 2018

Not sure, seems to be used for jenkins and not much more? At least from what I was finding some days ago.

I then looked into how to write Visual Studio Code plugins. Which would involve a local javascript server, as they don't support other languages right now, others are planned. Not sure if wrapping python would even be an option.

@mzfr
Copy link
Contributor

mzfr commented Mar 6, 2018

I personally use Sublime Text and other people might use different editors and creating a linter for every editor would be tough.

Perhaps we should look into creating a linter that can somehow be integrated with the Language Server Protocol? This will mean that we just have to write code once (hopefully in Python) and we'd use existing LSP plugins for each editor to get linting capabilities!

Let me know if this sounds okay, and I'll research this a bit more!

@razzeee
Copy link
Member

razzeee commented Mar 6, 2018

That sounds like the thing I was looking for. Please do some research, but don't get into deep.
Might be (a very interesting) gsoc project on it's own.

@razzeee
Copy link
Member

razzeee commented Mar 11, 2018

@enen92 do you also agree that the language server protocol would be the better way?

@enen92
Copy link
Member Author

enen92 commented Mar 11, 2018

@razzeee I am not familiar with it. From what I have seen it looks too complex for what we want to achieve, it almost feels like we are reinventing the wheel. We are not defining a new language and most of those features are already available for python. Even when developing for kodi, having kodistubs in site-packages already gives auto-completion as well. IMHO the idea would be to export the issues via a common data model that could later be consumed (and processed) by third party projects. That's why I suggested checkstyle which basically is the only one I know about.

@MartijnKaijser MartijnKaijser added this to To do in Roadmap via automation Oct 1, 2018
@MartijnKaijser MartijnKaijser moved this from To do to Done in Roadmap Oct 1, 2018
@MartijnKaijser MartijnKaijser moved this from Done to To do in Roadmap Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
  
To do
Development

No branches or pull requests

4 participants