search

kiegroup / github-action-build-chain

86 stars 24 forks source link

Fix head field used by octokit.rest.pulls.list() method #464

Closed rgdoliveira closed 6 months ago

rgdoliveira commented 6 months ago

In octokit/rest 18 doc (https://octokit.github.io/rest.js/v18) there is explanation about octokit.rest.pulls.list and for the head field it mentions:

Filter pulls by head user or head organization and branch name in the format of user:ref-name or organization:ref-name. For example: github:new-script-format or octocat:test-branch.

We are currently using user/fork-name:ref-name and from the documentation it seems it should be user:ref-name

rgdoliveira commented 6 months ago

e2e-regression-cli tests are failing, investigating why...

rgdoliveira commented 6 months ago

so those e2e-regression-cli tests are failing already and it is not related to my PR, see https://github.com/kiegroup/github-action-build-chain/pull/465. I will merge this one for now and can look later on those test failures.