PavanMudigonda / jacoco-reporter

GitHub Action to Publish JaCoCo Format Code Coverage XML and attach it to the Workflow Run as a Check Run. Also can enforce Coverage Quality Gate.
https://linkedin.com/in/PavanMudigonda
MIT License
63 stars 7 forks source link

bug in output variable coverage_percentage #21

Closed PavanMudigondaTR closed 2 years ago

PavanMudigondaTR commented 2 years ago

Describe the bug

Fix bug in formaule which is causing wrong calculation in output variable coverage_percentage

wrong output coming out of below due to string vs int. Need to convert to int before running formula $coveragePercentage = [math]::Round(100 - (($missedLines / $coveredLines) * 100))

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

PavanMudigondaTR commented 2 years ago

fixed in v4.4