OCA/oca-github-bot

Merging PR should mark a line in the "Migration to version xx" issue as done.

Closed this issue · 2 comments

legalsylvain commented

Ref : talk here with @pedrobaeza

Event : on pull_request / merged

Action :

  1. check in all the comments of the PR if one contains the order /ocabot migration yyy
  2. if yes, find the issue named "Migration to version xx"
  3. if found, find the line name "- [ ] yyy - By @xxx - PR ref"
  4. if found replace "- [ ]" by "- [x]"

Regarding step 1, I think it's going to be a little heavy to do that, but I don't see any alternative to identify the yyy (module name).

Comments welcome.

pedrobaeza commented

Instead of 1, I think it's better to just go to the migration issue of the target branch, and look for a line with the PR number. If found, mark it if not already done.

legalsylvain commented

Instead of 1, I think it's better to just go to the migration issue of the target branch, and look for a line with the PR number. If found, mark it if not already done.

💯 Indeed ! Thanks.