search

romeovs / lcov-reporter-action

Comments a pull request with the code coverage generated by your tests.
MIT License
161 stars 99 forks source link

Option `delete-old-comments` set to `true` does't delete old messages #46

Open ivanzusko opened 2 years ago

ivanzusko commented 2 years ago

Firstly, thank you for such a cool action 👍

I've faced with the problem, that the delete-old-comments: true option doesn't delete the old messages:

image

Configuration:

- name: Post coverage report
        uses: romeovs/lcov-reporter-action@v0.3.1
        with:
          lcov-file: "${{ env.WORKING_DIRECTORY }}/coverage/lcov_head.info"
          lcov-base: "${{ env.WORKING_DIRECTORY }}/coverage/lcov.info"
          github-token: "${{ secrets.SA_GITHUB_TOKEN }}"
          delete-old-comments: true
pcothenet commented 2 years ago

Looks like the same issue as #47

fredrikaverpil commented 1 year ago

You can use the same workaround as I posted in #47