search

JetBrains / ReSharper-InspectCode

C# static analysis on GitHub Actions using JetBrains ReSharper InspectCode.
Other
16 stars 1 forks source link

Include filter throws errors #8

Open OscarAtFatshark opened 3 months ago

OscarAtFatshark commented 3 months ago

Hi!

I am trying to get the include option to work for our Github projects, but it's not behaving according to specifications from what I can tell, and is not consistent with how it works locally when I run on my Windows 10 machine using the 2024.1.3 version (same as I use for the Github action).

I'm trying to join several files for the include argument using the ';' delimiter, but I get a complaint from the plugin when it parses it, see image below. image

I've also tested with forward slashes, and this gives a different error image

If I just run the same thing in Windows 10 on my machine in Powershell jb inspectcode -o="testie.txt" -f=Text --no-build --include="**\Test2.cs;**\test1.cs" ".\WpfApp1\WpfApp1.sln" I get no issues whatsoever image

Attached is the output of a full run of the step with backward slashes in Github.

jb_inspectcode_action_failure.txt

If I am doing it the wrong way somehow, how is this supposed to be done?

Best regards, Oscar

derigel23 commented 3 months ago

Wrap them in quotes?

OscarAtFatshark commented 2 months ago

Sorry for slow reply, been off for a while.

If I try "**\Test2.cs;**\test1.cs" it fails as well, so it doesn't make any difference what I can tell.

image

This is how the command line call looks. Seems like perhaps the quotation marks aren't propagated to the MODIFIED_FILES variable, not sure if that is significant or not. image

OscarAtFatshark commented 1 month ago

@derigel23 is there any progress on this, or do you have any other ideas I can try out? Currently preventing me from using it in our pipeline unfortunately.