Add GitHub Markdown test report templates#162
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
huzaifaalmesbah
left a comment
huzaifaalmesbah
left a comment
There was a problem hiding this comment.
I think we don’t need any changes on the combined-report page now because we’ve already moved this page to the archive.
Please revert the changes made to: archive/archived-test-reports/combined-report.md
There was a problem hiding this comment.
Agree with @huzaifaalmesbah , since the combined report page is already archived, I think we can revert the changes made to archive/archived-test-reports/combined-report.md and keep this MR focused only on active templates.
|
@yusufhay, are you available for this PR? Nikunj and I have requested changes. Please review and implement them. Once this is done, we can merge the PR. |
|
@yusufhay Just a friendly ping on this PR. Let us know if you need any help with the requested changes. Happy to assist! 🙂 |
The combined report page has been moved to the archive, so the new GitHub Markdown template additions are no longer needed here. Reverts the changes to archive/archived-test-reports/combined-report.md from the test report templates PR.
|
|
||
| ### Actual Results | ||
|
|
||
| - ✅ Expected result #1 works as expected with patch. |
There was a problem hiding this comment.
In the numbered text appears underlined. Could you please check that?
There was a problem hiding this comment.
Maybe it's from markdown format. It's not a major issue. i see all okay. Once this PR is merged, we can start work on the new proposal template.
| ### Actual Results | ||
|
|
||
| - ✅ Expected result #1 works as expected with patch. | ||
| - ✅ Expected result #2 works as expected with patch. |
There was a problem hiding this comment.
In the numbered text appears underlined. Could you please check that?
3 participants
Summary
Testing
Fixes #3