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

[report-converter] Support Roslynator #3765

Merged
merged 13 commits into from
Oct 24, 2022
Merged

Conversation

PeterKaszab
Copy link
Contributor

Closes #3764

@PeterKaszab PeterKaszab marked this pull request as ready for review October 16, 2022 15:09
@mcserep
Copy link

mcserep commented Oct 20, 2022

@dkrupp This is the C# Roslyn report converter I was talking about previously. Thank you if the CodeChecker team could find the time to review it.

@whisperity whisperity added tools 🛠️ Meta-tag for all the additional tools supplied with CodeChecker: plist2html, tu_collector, etc. other-analyzer 💬 Analysers which CodeChecker offers only report conversion of, but no driving report-converter 🔄 labels Oct 21, 2022
@bruntib
Copy link
Contributor

bruntib commented Oct 21, 2022

Please, fix the tests, but otherwise looks good to me.

@PeterKaszab
Copy link
Contributor Author

Please, fix the tests, but otherwise looks good to me.

@bruntib
Done in b4754da

@bruntib bruntib merged commit 6907145 into Ericsson:master Oct 24, 2022
@whisperity whisperity added this to the release 6.21.0 milestone Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
other-analyzer 💬 Analysers which CodeChecker offers only report conversion of, but no driving report-converter 🔄 tools 🛠️ Meta-tag for all the additional tools supplied with CodeChecker: plist2html, tu_collector, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Roslynator (C#) support to report-converter
4 participants