dorny / test-reporter

Displays test results from popular testing frameworks directly in GitHub
MIT License
839 stars 212 forks source link

Disabling git ls-files #169

Open hojalot opened 2 years ago

hojalot commented 2 years ago

We have a repo with over 25k files and the output of "git ls-files", possibly from dist/index.js call to listFiles(), adds significant noise to our logs. Is there a way to disable this feature?

sergeidyga commented 2 years ago

+1. We run tests from jar and don't need "git ls-files". But obviously - "git ..." also fails (since there was no checkout step before)

kudos3939 commented 1 year ago

We want to disable "git ..." too. Because of this, we cannot run the action in a separated yaml file through artifacts since it requires a git repository.

mesporas-talis commented 1 year ago

This issue addresses the git ls-files step: https://github.com/dorny/test-reporter/issues/210 Max annotations needs to be set to 0 when using the action.

However, it does not create annotations, but I saw that with the java-junit reporter. I'm not sure how it is with other reporters.

Example

- uses: dorny/test-reporter@v1
      with:
        artifact: test-results            # artifact name
        name: JEST Tests                  # Name of the check run which will be created
        path: '*.xml'                     # Path to test results (inside artifact .zip)
        reporter: jest-junit              # Format of test results
        max-annotations: 0
jon-nfc commented 2 weeks ago

+1 for the removal of git ls.

I've had to specifically add a actions/checkout@v4 step so that the report can be created in a job that just builds reports and doesn't require the repo checked out.