Comments (13)
As demonstrated by the negative reaction to a single ping here that might not have even actually happened, it's going to be very much an uphill battle trying to get engagement and consensus on a significant overhaul of the org setup just so that people can get even more pings (or the same amount of pings but done differently). I think this is the wrong approach and that we should abandon CODEOWNERS, or at least greatly reduce the number of files and teams in the CODEOWNERS file.
I think the problem is three-fold:
- People think they want fewer pings but also to be pinged only on things that they care about.
- There is no reliable way to programatically determine what PRs people will want to be pinged about. They might think there is ("I want all the changes to these files") but that leads us to the last problem....
- People think they know what they want but are sometimes not good at knowing what they want.
I think the CODEOWNERS file was a well-intentioned experiment that good people put work into but, unfortunately, has not worked out. There's nothing wrong about that. Failed experiments are part of the process. If all experiments succeed, they're probably not really experiments.
I think people generally ignore the CODEOWNERS ping. I could be wrong, but I think it's mostly noise at this point and we should get rid of it or at least vastly reduce its scope. Most files do not need CODEOWNERS entries.
This is related to the fact that the typical team contains one or two active people and six or eight people that will never respond to a ping and aren't involved in the org anymore. The only real solution to that is for one or more TSC members to actively manage the teams, removing a few people (and maybe adding a few) every few days (because doing all the teams in one sitting is just not possible).
from admin.
No one was pinged (except for @Trott)
I think everyone is pinged when you mention
@nodejs
I very much doubt it, GitHub docs says it's only when mentioning people or teams (https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#mentioning-people-and-teams), nothing about mentioning orgs. Also the notification I've received says subscribed
, not mention
nor team mention
.
from admin.
No, it can work by requesting reviews from individual instead of teams. The bot can check the CODEOWNERS file, list the members of the listed teams, and request reviews from them instead of pinging the team.
from admin.
Please be mindful of pinging anyone in the org for an administrative issue like this
from admin.
No one was pinged (except for @Trott)
from admin.
No one was pinged (except for @Trott)
I think everyone is pinged when you mention @nodejs
from admin.
I think everyone is pinged when you mention
@nodejs
I've tested it, and that's not the case (phew!)
from admin.
significant overhaul
I wouldn't call it significant, it's just moving teams under a single roof, and possibly changing the permissions of that house.
The benefit of a CODEOWNERS ping rather than the comment is that it doesn't subscribe them to the issue (IIRC). It lets them know they've been pinged, but doesn't subscribe them to future updates.
Whether or not CODEOWNERS is the best method to ping users, I think the comment is too much: it pings and subscribes users to the issue.
I think this is the wrong approach and that we should abandon CODEOWNERS, or at least greatly reduce the number of files and teams in the CODEOWNERS file.
If that's the case, the bot ping can be removed, and the CODEOWNERS file deleted.
In my opinion, the order of most pings to least pings is
3. Bot comment
2. CODEOWNERS review
\1. None
If the ping was to be kept, I think doing it using GitHub reviews would be better, but if the ping was to be removed, than that can all be ignored.
The benefit of removing the ping entirely is that organization members monitor the issues, so if someone needed to be pinged, it could be done manually by members.
However, I still think all teams should fall under the all-members team, for ease-of-management.
And, if the all-members team is given triage access (or explicit read might work), members can request reviews from teams when needed.
from admin.
Ideally getting notified when some files are modified would be a feature of GitHub, so folks do not need to ask for someone to add them to a team (and remove them when they no longer want the notifications), and we do not need to manage teams at all (except those who give write access to some repo). Having said that, I realize it's convenient to be able to ping teams in e.g. issues, and we might miss having teams in those cases.
Realistically, we have teams, and have to use them (and we the TSC should do a better job at managing them).
Regarding the proposal of giving triage permission to all-members
: -1, that would force us to either gate-keep joining ping-only teams, and force us to "clean up" inactive members.
Regarding moving more teams to all-members
: why? why do you care? why should we care?
Regarding replacing bot pings with bot requesting reviews: why not, that seems relatively easy to do, I've seen it done on other open-source repo, I'd say it's a good idea, let's go for it (but it's not an issue for admin
, it only requires someone to open a PR).
Regarding giving the possibility to request review from teams instead of individuals: meh, not worth it IMO.
from admin.
Regarding replacing bot pings with bot requesting reviews: why not, that seems relatively easy to do, I've seen it done on other open-source repo, I'd say it's a good idea, let's go for it (but it's not an issue for admin, it only requires someone to open a PR).
Regarding giving the possibility to request review from teams instead of individuals: meh, not worth it IMO.
The reason this is an issue for admin, is because all teams need to be under all-members (or each team needs to have added permissions) for this to work. Once all teams are under all-members, that team needs to be given triage permissions (which is the minimum needed for reviews on teams)
from admin.
No, it can work by requesting reviews from individual instead of teams. The bot can check the CODEOWNERS file, list the members of the listed teams, and request reviews from them instead of pinging the team.
Got it. I can open a PR later today. Feel free to close this if you think it's resolved, but otherwise feel free to keep this open.
from admin.
Sorry for the noise, I've moved this to nodejs/node#53897
from admin.
Closing in favor for discussion there.
from admin.
Related Issues (20)
- New labels for `nodejs/help`
- Create repository nodejs/ambassadors HOT 3
- The Future of nodejs/examples? HOT 18
- Creating a new nodejs/nbytes repo HOT 5
- Temporay admin access for @bensternthal HOT 3
- Give Triagers More Control Over Issues HOT 2
- Transfer elastic/require-in-the-middle to the org HOT 8
- Enable GitHub fine-grained personal access tokens HOT 2
- Transfer amaro to the org HOT 5
- Remove coverage-admins team? HOT 7
- Remove dtrace-mdb team? HOT 8
- Remove the social-media-delegates team HOT 2
- Create nodejs/typescript repo HOT 9
- Conversion to Enterprise account HOT 32
- Becoming maintainer of llhttp and its related repositories HOT 24
- Grace Hopper Open Source Day - Help Needed from our Node.js Community!
- add `llparse` to team `llhttp`, request @indutny to transfer llparse-builder HOT 3
- Enable github actions can create pull requests HOT 6
- Remove Medium email account HOT 4
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 admin.