Skip to content
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

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

ankitguptaindia
Copy link

@ankitguptaindia ankitguptaindia commented Feb 7, 2024

Submitting the PR for adding a Test Handbook page for Profile Badge details.

Closes: #19

Copy link
Collaborator

@ironprogrammer ironprogrammer left a 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.

test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
ankitguptaindia and others added 8 commits February 21, 2024 17:57
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>
@ankitguptaindia
Copy link
Author

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

Copy link
Collaborator

@ironprogrammer ironprogrammer left a 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.

test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@ironprogrammer ironprogrammer left a 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.

test-team-badge.md Outdated Show resolved Hide resolved
test-team-badge.md Outdated Show resolved Hide resolved
ankitguptaindia and others added 4 commits March 28, 2024 17:36
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>
test-team-badge.md Outdated Show resolved Hide resolved
ankitguptaindia and others added 5 commits March 28, 2024 18:26
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
@ironprogrammer
Copy link
Collaborator

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 👍🏻

@pfefferle
Copy link
Member

Sorry for being late, but what about the Hosting-Test-{Runner|Reporter}:

@ironprogrammer
Copy link
Collaborator

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?

Copy link
Collaborator

@ironprogrammer ironprogrammer left a 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.

test-team-badge.md Show resolved Hide resolved
Co-authored-by: Brian Alexander <824344+ironprogrammer@users.noreply.github.com>
@ankitguptaindia
Copy link
Author

@ankitguptaindia As this hasn't been merged, I've added one more item here. Props @pfefferle.

@ironprogrammer This new criteria has been merged and added to the handbook page. Thanks @pfefferle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants