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

Expose extra command line options #420

Closed
seanm-fathomhq opened this issue Oct 26, 2023 · 5 comments
Closed

Expose extra command line options #420

seanm-fathomhq opened this issue Oct 26, 2023 · 5 comments

Comments

@seanm-fathomhq
Copy link

Would it be possible to expose the --no-build and the --caches-home options? I am working in a large solution and trying to bring down the run time of the action with caching and it's a bit tricky at the moment because these options aren't exposed.

@muno92
Copy link
Owner

muno92 commented Oct 29, 2023

@seanm-fathomhq

Add --no-build option and --caches-home option in version 1.9.0.
https://github.com/muno92/resharper_inspectcode/releases/tag/1.9.0

You can use it as follows.

Example)

on: push

jobs:
  inspection:
    runs-on: ubuntu-latest
    name: Inspection
    steps:
      - name: Checkout
        uses: actions/checkout@v3
      - name: Setup .NET
        uses: actions/setup-dotnet@v3
        with:
          dotnet-version: '7.0.x'
      - name: Restore
        run: dotnet restore
      - name: Inspect code
        uses: muno92/resharper_inspectcode@v1
        with:
          solutionPath: ./YourSolution.sln
          noCache: true
          cachesHome: /path/to/caches_home

@seanm-fathomhq
Copy link
Author

Perfect, thank you 😀

@seanm-fathomhq
Copy link
Author

Whoops, reopening this as I only just got to testing out the new options. It looks like the noBuild option is broken, returning the error Specify either '--build' or '--no-build'. Looking at your code, the base command includes the --build option, and later if noBuild is true it appends --no-build but doesn't remove --build so it clashes. Probably need to remove --build from the base command and add it in an else when checking noBuild

@seanm-fathomhq seanm-fathomhq reopened this Nov 8, 2023
@muno92
Copy link
Owner

muno92 commented Nov 8, 2023

@seanm-fathomhq
Oh, sorry!

I forgot --build option was already specified, and fixed it in version 1.9.1.

@seanm-fathomhq
Copy link
Author

All good 😁 I should have tested it before closing to make sure. I have definitely tried it this time now though and it is working, thank you!

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

2 participants