ci: split righlib sync into create-pr and check-pr commands #13583
Conversation
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
.github/workflows/rightlib_sync.yml
Outdated
| ./sync-rightlib.py create-pr | ||
|
|
||
| - name: check-pr | ||
| if: github.event.schedule == '30 1-23 * * *' || (github.event.inputs && github.event.inputs.command == 'check-pr') |
There was a problem hiding this comment.
Давай два отдельных wf сделаем? Меня очень смущает, что закладка на расписание (кто то поменяет расписание -> забудет здесь поменять -> все сломается)
Типа просто сделать воркфлоу "Autocomplete PR" и в нем комплитить ПРы (в пределе может не только иморты из аркадии)
There was a problem hiding this comment.
ну тут либо три отдельных, либо action. я сделал с action сейчас, но мне не нравится.
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
I refactored rightlib sync script to split the run logic into two commands:
create-prandcheck-pr, and updated the workflow accordingly.Also, I changed schedule start minute to 30, becasuse according to docs start of every hour is high load times and our can be delayed or dropped.
Changelog entry
...
Changelog category
Additional information
...