Updates not-today.md to add github-handle var for Jacquelin Coronilla#8140
Conversation
|
Want to review this pull request? Take a look at this documentation for a step by step guide! From your project repository, check out a new branch and test the changes. |
andyvu923
left a comment
There was a problem hiding this comment.
Hi @ryanfkeller , thanks for working on this issue!
Things done well
- Correct branches are used
- PR links correct issue
- Code changes are relevant and clean
Suggested Changes
No changes necessary for this PR. Only thing to mentions is that backticks ( ` ) can be used to enclose code variables or file names, such as not-today.md becoming not-today.md
Thanks for fixing this issue. PR Approved 👍👍
|
Availability: after 1pm (pacific) |
kdaca19xx
left a comment
There was a problem hiding this comment.
Thank you, ryanfkeller!
Things Done Well
-
The pull request done with the correct branch.
-
There's a linked issue (and I understand it).
-
The change in the Files Changed tab looks good.
-
I appreciate your mention of the file, not-today.md.
Suggestions
none.
Fixes #7782
What changes did you make?
Why did you make the changes (we will use this info to test)?
CodeQL Alerts
Instructions for resolving CodeQL alerts
If CodeQL alert/annotations appear, refer to How to Resolve CodeQL alerts.
In general, CodeQL alerts should be resolved prior to PR reviews and merging
Screenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes)