mikepenz / action-junit-report

Reports junit test results as GitHub Pull Request Check
https://blog.mikepenz.dev
Apache License 2.0
305 stars 116 forks source link

Suggestion: provide info on setting --max_old_space_size #1170

Closed arcuri82 closed 2 weeks ago

arcuri82 commented 2 weeks ago

Hi, thx for this action. Recently, it started to fail due to:

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory

I have thousands of tests...

the solution was to use max_old_space_size, eg, currently I am using:

      - name: Publish Test Report
        if: success() || failure()
        uses: mikepenz/action-junit-report@v4
        env:
          NODE_OPTIONS: "--max_old_space_size=4096"
        with:
          report_paths: '**/target/surefire-reports/TEST-*.xml'

maybe something worthy to mention in the documentation, in case other users end up in the same problem.

This is base on this discussion.

mikepenz commented 2 weeks ago

Thank you very much for the issue.

This is already outlined in the README:

Screenshot 2024-08-30 at 16 03 00 Screenshot 2024-08-30 at 16 03 23

Will see to adjust the wording to make it easier to find

arcuri82 commented 2 weeks ago

Hi @mikepenz

you are right, indeed it was there. But, as by default it is collapsed, I missed when "quickly" scan through the docs. Didn't realize could click on those arrows to expand the text...

mikepenz commented 2 weeks ago

No worries. I hope with the update if the readme it's easier to find 👍