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

Output InspectCode result in XML format #1022

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

pascalberger
Copy link
Member

Latest InspectCode version generates a SARIF file by default. Enforce to write an XML file which Cake.Issues can parse.

Once we've updated to Cake Issues 3.x we can use the available SARIF parser.

Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@gep13 gep13 merged commit ec6f85b into cake-contrib:develop Aug 5, 2024
4 checks passed
@gep13
Copy link
Member

gep13 commented Aug 5, 2024

@pascalberger thank you for catching, and fixing, this!

cake-contrib-bot pushed a commit that referenced this pull request Aug 5, 2024
Merge pull request #1022 from pascalberger/feature/inspectcode-xml

Output InspectCode result in XML format
@pascalberger pascalberger deleted the feature/inspectcode-xml branch August 5, 2024 10:25
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

Successfully merging this pull request may close these issues.

Reference to tool JetBrains.ReSharper.CommandLineTools in toolsettings.cake needs to be updated
2 participants