Comments (1)
@jberkus:
This request has been marked as needing help from a contributor.
Guidelines
Please ensure that the issue body includes answers to the following questions:
- Why are we solving this issue?
- To address this issue, are there any code changes? If there are code changes, what needs to be done in the code and what places can the assignee treat as reference points?
- Does this issue have zero to low barrier of entry?
- How can the assignee reach out to you for help?
For more details on the requirements of such an issue, please see here and ensure that they are met.
If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help
command.
In response to this:
Migrated from the original issue.
What would you like to be added: It would be nice to have trigger include a bold, explicit invitation to join the org in its ok-to-test message after you've made a three PRs (that is, on your fourth), sample message:
We noticed you've done this a few times! Consider joining the Kubernetes org to skip this step, as well as gain LGTM and other bot rights rights. We recommend asking the approvers on your prior PRs to be your sponsors.
(It would also be nice if we had a document to link to that wasn't all about reasons you can't be a member...)
We can do a query like
type:pr is:merged org:kubernetes author:$user
before making the comment to determine if someone is qualified - this graphql query has a flat cost of 1:query { search(query: "type:pr is:merged org:kubernetes author:katharine", type: ISSUE) { issueCount } }Alternatively, we can just always comment and leave the user to figure out whether they count, but I think this diminishes the effectiveness over a positive assertion.
If there are issues with doing a count over the whole org, then doing it just on the current repository would be adequate.
Why is this needed: Joining the org is pretty intimidating. On top of that, it is wildly unclear when people are eligible to join, and even if they clearly are, it often doesn't occur to them to actually do so. An estimated 1/3 of our regular contributors are not org members. While this doesn't help contributors who do not make PRs who would be eligible for org membership, it would be helpful to those who do. The real bar is much lower than non-members tend to think it is.
We currently mention that "regular contributors should join the org to skip this step", but it's not at all noticeable and "regular contributors" is wildly open to interpretation.
(shoutout to @liztio for the concept, and @Katharine for the original writeup)
/kind feature
/sig contributor-experience
/priority backlog
/help
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.
from prow.
Related Issues (20)
- Allow renaming Prow's `test` container for individual jobs HOT 3
- prow: handle the case of re-triggering an expired GitHub workflow HOT 1
- Prow issue: Bad invalid commit message for npm dependencies #28257 HOT 2
- Allow `ok-to-test` label to approve GitHub workflow runs for new contributors #25210 HOT 3
- Reporting periodic jobs failures to slack channel after number of consequent failures HOT 2
- Prow seems to assign Assignees who are not in OWNERS file HOT 3
- Prow web UI exposes private repositories HOT 5
- plugins: do we still need the `cla` plugin? HOT 2
- [BUG] CC multiple people does not work anymore HOT 13
- Configured Tests Dashboard HOT 1
- Branch protection rules prevent first PR of a merge-batch from being merged
- Crier failing to report OOMed pods HOT 9
- switch to aws-sdk-go-v2 as v1 is going away HOT 1
- Prow issue:
- prow config: where possible, all config should behave like branchprotector for overrides
- Prow issue: HOT 2
- Git providers other than GitHub HOT 1
- Tide re-ran an overridden job HOT 2
- As an unprivileged user, I can write to prow on the server side. HOT 2
- prowjob CRD is too large to apply HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from prow.