MishaKav / pytest-coverage-comment

Comments a pull request with the pytest code coverage badge and full report
MIT License
181 stars 60 forks source link

Adding in support for workflow_dispatch events: Take Two #149

Closed alexjyong closed 1 year ago

alexjyong commented 1 year ago

This PR will allow this action to be used on workflow_dispatch events. The user will be required to pass a PR number via issue-number in the inputs, but this will allow folks to get coverage reports on demand in a github action instead of having to tie it to a pull-request/push event.

I needed this for a work project, so I figured I'd share this with everyone else!

(Re do of https://github.com/MishaKav/pytest-coverage-comment/pull/143 )

alexjyong commented 1 year ago

@MishaKav I pulled in the latest changes from main on your repo and fixed up the issues you had: https://github.com/MishaKav/pytest-coverage-comment/pull/149

MishaKav commented 1 year ago

I'm so sorry for bothering you. I still can't build your version. Try to run npm run all after your fix.

image
alexjyong commented 1 year ago

@MishaKav no worries. Strange, I tested this and it worked on my repo. (unfortunately private so I can't share.)

I'll take a look.

alexjyong commented 1 year ago

@MishaKav I had issues with that branch, so I made my PR based off of main instead of that branch. I'll take a closer look though. No idea if that's related though.

alexjyong commented 1 year ago

@MishaKav updated this on: https://github.com/MishaKav/pytest-coverage-comment/pull/150