Skip to content

[19.0][FIX] generate-analysis: use OCA git bot as author and committer - #6074

Merged
pedrobaeza merged 1 commit into
OCA:19.0from
hbrunn:19.0-upgrade_analysis-bot-user
Nov 3, 2026
Merged

pedrobaeza merged 1 commit into
OCA:19.0from
hbrunn:19.0-upgrade_analysis-bot-user

Conversation

@hbrunn

@hbrunn hbrunn commented Nov 3, 2026

Copy link
Copy Markdown
Member

as the generated committer/author mail addresses interfere with cla bot, we need to set some known good address here. generated PRs then look like this.

after merging, we'll have to rerun the analysis job manually to get the PRs updated.

@legalsylvain legalsylvain left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks !

@pedrobaeza pedrobaeza added this to the 19.0 milestone Nov 3, 2026
@pedrobaeza
pedrobaeza merged commit c15e796 into OCA:19.0 Nov 3, 2026
8 checks passed
@hbrunn

hbrunn commented Nov 3, 2026

Copy link
Copy Markdown
Member Author

I've triggered a rerun of the analysis job

@MiquelRForgeFlow

Copy link
Copy Markdown
Contributor

Shouldn't this be backported to 18.0 too?

@hbrunn

hbrunn commented Nov 3, 2026 •

Copy link
Copy Markdown
Member Author

we don't really need it because the main branch run the analysis for all versions. but I'll cherry-pick it into v20 which probably needs some changes anyways due to different dependencies. I'll probably rewrite the whole thing to use the OCA docker images instead of a manual installation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants