Skip to content

Conversation

@CheaterCodes
Copy link
Contributor

@CheaterCodes CheaterCodes commented Dec 28, 2021

Rendered Views:

The following changes were made;

  • Removal of technical teams
  • Removal of technical leads
  • Addition of team leads
  • Generalization of team responsibilities
  • Reformating of team creation RFC template
  • Definition of sub teams
  • Redefinition of the administrative board
  • Change of admin election process
  • Incomplete redefinition of processes

I'd like to see more done for standard processes, possible more in line with the Community Team RFC.

- Removal of technical teams
- Removal of technical leads
- Addition of team leads
- Generalization of team responsibilities
- Reformating of team creation RFC template
- Definition of sub teams
- Redefinition of the administrative board
- Change of admin election process
- Incomplete redefinition of processes
@TheGlitch76
Copy link
Contributor

We currently have a subteam that we don't consider full Quilt Developers (Mappings Triage). They don't have the Quilt Developer role, or (as far as i'm aware) access to our internal dev chat or meetings.

We should clearly define this relationship, or make Mappings Triage members also "Quilt Developers"

@triphora
Copy link
Contributor

triphora commented Jan 6, 2022

Does this address anything regarding the addition of staff? I could find removal of staff, but nothing about addition.
On another note, I think that the Community Team RFC (0007) could be generalised a bit more in alignment with RFC 6. For example, the keyholder is specified in the community team RFC for some reason, and the general idea of our vote system is rather similar to that specified here, just with some changed specifics.

@triphora triphora mentioned this pull request Jan 8, 2022
@TheGlitch76
Copy link
Contributor

TheGlitch76 commented Jan 13, 2022

Addition of staff is very specific to the team, and isn't defined for the organization as a whole on purpose IMO.
Most developer teams "elect" members by a quick voice vote (@Team is it ok if we add John Doe in #dev-chat), but some can be joined by anyone already on a parent team (QSL subteams), or are appointed by a parent team instead of self-managing (Mappings Triage).
The Community Team has a completely different, well-defined process for addition of staff that is intentionally long and difficult.

@ToffeeMax
Copy link
Contributor

I am generally content with the changes put forward by this PR. There would need to be a review of the core documents to make sure this does not cause any clashes -- but it sees you have done this to a good extent.

I do think this would be a fantastic chance to also clarify the way that PRs are merged overall -- which is outlined in RFC-0001 and make it crystal clear. As sometimes I cannot even recall the general method: Possibly specifying the team responsible. In this case: this would be a team wide approval

Add new, detailed process for the election of admin board members.
OroArmor pushed a commit that referenced this pull request Mar 1, 2022
* RFC 48: Communications Team

This is a proposal for a new team separate from the community team or developer teams. It is built on top of the process laid out for creating new teams in #47.

* Update 0048-communications-team.md

* Update and rename 0048-communications-team.md to 0048-outreach-team.md

* Update 0048-outreach-team.md
@Akarys42
Copy link
Contributor

This PR has been going for about a year and a half now, and has virtually been in effect for a year already. Team Leads are already implemented, two Admin Board votes have been performed according to this RFC, and probably some other things I'm not remembering.

I would suggest to do one last round of fixing, and put this RFC into FCP. It seems clear so far there are no fundamental issues with it, and the bikeshed nightmare it has ended up in is quite detrimental to the project.

Copy link
Contributor

@Akarys42 Akarys42 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 commented on the discrepancies that exist between the version suggested by this PR and what's actually applied currently

@Akarys42
Copy link
Contributor

All outstanding comments on this RFC have been resolved. Sadly, I cannot actually mark them as resolved due to GitHub limitations.

My plan is to request an FCP for this PR this weekend.

Copy link
Contributor

@gdude2002 gdude2002 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I heard ya like reviews.

It might be worth getting the policy work done on better defining what the admin board does first, before merging this - such policy work seems likely to require changes to this RFC as well.

@Akarys42
Copy link
Contributor

@gdude2002 consider yourself re-requested for review :P

Ambre Bertucci and others added 3 commits March 24, 2023 19:22
Copy link
Contributor

@gdude2002 gdude2002 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have some more changes!

@OroArmor OroArmor requested a review from gdude2002 April 4, 2023 22:34
@Akarys42
Copy link
Contributor

This PR is placed under FCP, lasting until April 25th.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

final-comment RFCs in the final comment period.

Projects

None yet

Development

Successfully merging this pull request may close these issues.