Fix the outdated button name - #138
Foundsheep wants to merge 1 commit into
Conversation
|
Ah, actually the only one issue left open #137, whose title doesn't describe anything related to this subject in fact, is about this change. I can't edit the content above, so I'm just leaving this comment. |
bbarrio2
left a comment
bbarrio2
left a comment
There was a problem hiding this comment.
<h2 hidden>Game over! Want to play again?! Just click refresh. 🧑🚀!</h2>
|
|
||
| 1. In the pull request navigation, select the **Conversation** tab. | ||
|
|
||
| 1. Scroll down and click the **Commit suggestion** button to open a commit message form. |
There was a problem hiding this comment.
| 1. Scroll down and click the **Commit suggestion** button to open a commit message form. | |
| <h2 hidden>Game over! Want to play again?! Just click refresh. 🧑🚀!</h2> |
There was a problem hiding this comment.
@bbarrio2 I think you got the wrong place to do your exercise. You should do that on your own repository
|
I created the branch, pulled it, and did the steps. |
|
Great contribution! I reviewed the changes and the approach looks clear. I especially like the way you handled the implementation. One suggestion would be to consider adding more comments/documentation around the main logic to make it easier for future contributors to understand. Overall, nice work! |
|
@FidelusAleksander Hi, sorry for tagging you. Please assign this to anyone you know. Anway, can someone check this PR? I think this is quite a simple one and can be merged quickly. I think this one is drawing too much attention unnecessarily now... |
5 participants
Summary
Step 4 uses an outdated name of a button, and is causing a bit of confusion.
Changes
It says to click the
Commit suggestionbutton, but it is actually nowApply suggestion.This word has changed only.
Closes: Nothing. In the guideline, this type of simple fix can directly head to PR, so I didn't make any issue.
Task list