Associated PRs in task.yaml

[eyssartk]

This commit provides a new textarea to fill links to PRs used to complete the task.

[DBees]

@eyssartk As I understand, the intent of this "associated PR template change" is to ask the user to paste a link to a pull request into the task issue.
No problem with this intent but I'm not 100% sure it is what we want.

Note, there is already a method to create a link to a pull request (as discussed on mattermost CVA6 channel): "You can also go into the issue and link a pull request from there.To link it this way, you have to click the gear icon next to "Development" and edit the "link to pull request"." The advantage of the gear icon method is, I think, that you can then display linked pull requests on the project board. So, I think we would want users to use this method rather than simply pasting the URL into the issue.

(As far as I can see however, the gear icon method is only available after the issue is created which isn't ideal. )

I'm reluctant to experiment on these repos, but if you have some issues to link to PRs, you could try out the gear icon method and validate that is what we want. It may be the case you want BOTH to paste the URL, AND to link using the gear icon. I'm not sure.

[eyssartk]

@DBees I tried to reproduce the "link to a pull request" as discussed in the Mattermost but It seems access rights are different because I could not link a PR (no gear next to development) in CVA6 repo nor CVA6-SDK. Even after issue creation.

Also it seems that this linked PR produce a dependancy: closing a PR will close the task. Is this really what we want ? Surely a task can be fulfilled in multiple PRs. Can we link an issue to multiple PRs ?

[DBees]

Ok, to record what we found/discussed:

• With triage access as you (@eyssartk ) have, you still don't see the development gear icon for PR and Issues
• To be confirmed, but in any case CVA6 team doesn't want to automatically close issues linked from a PR
• Therefore, this pull request to adjust the Issue template with an informative link to PRs is appropriate.