-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test Team Profile Badge Process #20
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR, @ankitguptaindia! I've left some suggestions below.
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Thanks @ironprogrammer for the feedback, I have addressed all major feedback and created a post for getting feedback on these new criteria here: Test Team Profile Badge – Your Feedback Needed!. Thanks! cc: @webtechpooja |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added a few additional suggestions as this page content is shaping up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, one more little batch.
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
Clarifies that table lead qualification applies to Test table only
I hold on merging this, pending feedback on the drafted page from the Test team. Once the new page is published, we can wrap up this PR 👍🏻 |
The Page is live now https://make.wordpress.org/test/handbook/test-team-profile-badges/ 🎉 |
Sorry for being late, but what about the Hosting-Test-{Runner|Reporter}: |
To clarify, @pfefferle, are you suggesting that contributions to those two projects be qualification for a Test badge? If so, then YES, I agree it would be good to acknowledge that effort. The feedback provided by hosts from automated testing is also an essential ingredient to a healthy WP ecosystem. Participating hosts are highlighted in Make/Hosting, but perhaps the individual contributors who configure and run those for their hosts could also qualify (it's a lot of work to set up and maintain). I'm not sure how best to identify them, however. Thoughts? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ankitguptaindia As this hasn't been merged, I've added one more item here. Props @pfefferle.
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
@ironprogrammer This new criteria has been merged and added to the handbook page. Thanks @pfefferle |
Submitting the PR for adding a Test Handbook page for Profile Badge details.
Closes: #19