Skip to content
This repository was archived by the owner on Sep 1, 2022. It is now read-only.

Commit 5981876

Browse files
authored
Merge pull request #121 from githubtraining/fix-suggested-changes
Move away from octokit for suggested changes
2 parents a442720 + 3efe0a1 commit 5981876

File tree

2 files changed

+10
-14
lines changed

2 files changed

+10
-14
lines changed

config.yml

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -330,20 +330,10 @@ steps:
330330
- type: assignRegistrant
331331
issue: "%actions.newPullRequest.data.number%"
332332
## Suggest a change in the new PR
333-
- type: octokit
334-
method: pullRequests.createComment
335-
owner: "%payload.repository.owner.login%"
336-
repo: "%payload.repository.name%"
337-
number: "%actions.newPullRequest.data.number%"
338-
commit_id: "%actions.newPullRequest.data.head.sha%"
339-
path: game.js
340-
body: |
341-
How's this change?
342-
343-
```suggestion
344-
starCtx.fillStyle = "#000";
345-
```
346-
For more information about suggest changes, check out this [GitHub Help](https://help.github.com/articles/incorporating-feedback-in-your-pull-request) article.
333+
- type: createPullRequestComment
334+
body: suggested-change.md
335+
file: game.js
336+
pullRequest: '%actions.newPullRequest.data.number%'
347337
position: 5
348338
- type: requestReviewFromRegistrant
349339
pullRequest: Accidental bugs

responses/suggested-change.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
How's this change?
2+
3+
```suggestion
4+
starCtx.fillStyle = "#000";
5+
```
6+
For more information about suggest changes, check out this [GitHub Help](https://help.github.com/articles/incorporating-feedback-in-your-pull-request) article.

0 commit comments

Comments
 (0)