Add Copilot PR reviewer#8043
Conversation
github-actions
Bot
added
Build: Automation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
WaelAbuSeada
changed the title
| @@ -0,0 +1,1146 @@ | |||
| <# | |||
There was a problem hiding this comment.
I wonder if this is the right place to put an agent. But we can always move it, if need be. For now, I'll won't argue.
|
Let's get this agent on the road ASAP 🚀 |
mazhelez
left a comment
mazhelez
left a comment
There was a problem hiding this comment.
Left a couple of comments. There are also a couple of actions before the PR could be merged.
| name: Copilot PR Review | ||
|
|
||
| on: | ||
| pull_request_target: |
There was a problem hiding this comment.
You may want to limit the target branches:
E.g.
branches: [ 'main', 'releases/*' ]
| persist-credentials: true | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2 |
There was a problem hiding this comment.
There's v6.4.0. Any reason to use such an old version?
| & "${{ github.workspace }}/trusted/src/Tools/code review/scripts/Invoke-CopilotPRReview.ps1" | ||
|
|
||
| - name: Upload review artifacts | ||
| uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4.6.0 |
There was a problem hiding this comment.
Same here, the newest version is v7.0.1. Quite further ahead.
| @@ -0,0 +1,269 @@ | |||
| --- | |||
| name: al-code-review | |||
There was a problem hiding this comment.
I think this ( the "code review" folder) should live in .github, or ./tools, not under src/Tools, as that path is used for AL projects that act as tools in product.
Follow the convention to use capital case for folders.
5 participants
Ports the Copilot PR reviewer from BCAppsCampAIRHack into BCApps.
Added
Path adaptation
Fixes: AB#626808