search

massalabs / massa

The Decentralized and Scaled Blockchain
https://massa.net
5.57k stars 711 forks source link

Cannot merge external contributor PRs #4694

Closed AurelienFT closed 4 months ago

AurelienFT commented 4 months ago

External contributor PR such as https://github.com/massalabs/massa/pull/4686 can't be merged because auto-merge requires the whole CI pipeline to pass but the doc job never success on the PR depending on branches from fork. @Leo-Besancon can you take a look as you were the one that setup this (if it's not you can you redirect the issue) please ?

Leo-Besancon commented 4 months ago

I think @damip was the one to have access to the repo maintenance settings. Could you make the "docs" CI job not required for merging? (for example gas-costs-check is not required). It's probably the easiest, no?

AurelienFT commented 4 months ago

It seems to be ok, the auto merge works. Closing