dailymotion-oss / octopilot

Automate your Gitops workflow, by automatically creating/merging GitHub Pull Requests
https://dailymotion-oss.github.io/octopilot/
MIT License
174 stars 23 forks source link

feat: improve the display of repositories pr url at the end #307

Closed JordanGoasdoue closed 7 months ago

JordanGoasdoue commented 9 months ago

I've put it as a feature because it changes the behavior of octopilot

Let's take as example a repository where the PR is unrevised (the PR already exist, not yet merged and doesn't need new update)

INFO[0001] Updates finished                              repositories-count=1
INFO[0001] Update summary (Unrevised)                    unrevised-repository-count=1
INFO[0001] Update summary (Unrevised)                    unrevised-repository-pr-url="https://github.com/xxx/yyy/pull/777"

Let's take as example a repository where the PR is created / updated by octopilot

INFO[0001] Updates finished                            repositories-count=1
INFO[0001] Update summary (Updated)                    updated-repository-count=1
INFO[0001] Update summary (Updated)                    updated-repository-pr-url="https://github.com/xxx/yyy/pull/888"

For instance in dry-run you will be able to see already opened PR that doesn't need new changes compare to the one you want to apply.

WARN[0173] Running in dry-run mode, not pushing changes  repository=a/z

INFO[0173] Updates finished                              repositories-count=5
INFO[0173] Update summary (Unrevised)                    unrevised-repository-count=3
INFO[0173] Update summary (Unrevised)                    unrevised-repository-pr-url="https://github.com/a/b/pull/5"
INFO[0173] Update summary (Unrevised)                    unrevised-repository-pr-url="https://github.com/a/c/pull/41"
INFO[0173] Update summary (Unrevised)                    unrevised-repository-pr-url="https://github.com/a/d/pull/7"

It's also easier this way to list all the pullrequests and order them by type (unrevised vs updated)