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

LFDT updates for maintainers inactivity #292

Closed

Conversation

arsulegai
Copy link
Member

No description provided.

@arsulegai arsulegai changed the base branch from gh-pages to lfdt-updates August 29, 2024 07:06
@github-actions github-actions bot added quarterly-report Quarterly Project Updates meeting-minutes TOC Meeting Minutes labels Aug 29, 2024
@arsulegai arsulegai removed the meeting-minutes TOC Meeting Minutes label Aug 29, 2024
@@ -9,24 +9,24 @@ nav_order: 8

# Inactivity Policies

As part of the normal lifecycle of a project codebases and maintainers come and go. This document formalizes a Hyperledger wide policy for moving maintainers to emeritus status when a project does not have an explicit policy of their own, and the responsibilities of the Technical Advisory Council (TAC), Hyperledger Foundation staff, and maintainers in this process.
As part of the normal lifecycle of a project, codebases and maintainers come and go. This document outlines a LF Decentralized Trust-wide policy for moving maintainers to emeritus status when a project's TSC does not have an explicit policy of its own, as well as the responsibilities of the Technical Advisory Council (TAC), LF Decentralized Trust staff, and maintainers in this process.
Copy link
Member

Choose a reason for hiding this comment

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

Should the organizational policy be parallel with the project’s TSC policy, or only in effect when the TSC doesn’t have a policy? I think the latter might be too limiting.

Copy link
Member

Choose a reason for hiding this comment

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

I'd vote for it providing the defaults in case the TSC doesn't feel the need to define their own.
On Cacti we always try to go with the defaults so it would be great for us to preserve maintainer (TSC member) resources/time.

For those who would like to have more custom policies, this would still give them the option to define things for themselves.

It should be mentioned that some things from the organizational policy should be non-overridable, DCO being the example coming to mind.

Copy link
Member Author

Choose a reason for hiding this comment

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

@swcurran are you considering a case where the project may have lenient policy as opposed to being stricter?

Copy link
Member

Choose a reason for hiding this comment

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

I'm thinking of the case where the project is not enforcing their policy. I think even if they have a policy, this policy must be used. For example, when the inactivity in the Project's TSC itself.


When a maintainer has not had any activity in a particular project for three months they will receive a notification informing them of the inactivity policies. The means and manner of notification (email, github mentions, etc.) will be at the discretion of the TAC Chair or who the TAC Chair designates.
When a maintainer has not had any activity in a particular project for three months, they will receive a notification informing them of the inactivity policies. The means and manner of notification (e.g., email, GitHub mentions) will be at the discretion of the TAC Chair or their designee.
Copy link
Member

Choose a reason for hiding this comment

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

This should happen even if the TSC has a policy about removing maintainers. Further, the project (the TSC? the list of all maintainers?) should also receive the notifications about the lack of activity of maintainers.

Copy link
Member

Choose a reason for hiding this comment

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

(the TSC? the list of all maintainers?)

I'd say both.

This should happen even if the TSC has a policy about removing maintainers.

Plus one!
This should be one of those things that cannot be overridden from the organizational policy.


When a maintainer has not had any activity in a particular project for six months a proposal will be opened up to move the maintainer from active status to emeritus status. A member of the TAC or a Hyperledger staff member will open this proposal. Any permissions to approve pull requests or commit code and any other such privileges associated with maintainer status will be removed.
When a maintainer has not had any activity in a particular project for six months, a proposal will be opened to move the maintainer from active status to emeritus status. This proposal will be initiated by a member of the TAC or LF Decentralized Trust staff. All permissions to approve pull requests, commit code, and other related privileges will be removed.
Copy link
Contributor

Choose a reason for hiding this comment

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

I recommend that we add "a member of the project's TSC". It is probably more important that the project is responsible for this.


When a maintainer has not had any activity in a particular project for three months they will receive a notification informing them of the inactivity policies. The means and manner of notification (email, github mentions, etc.) will be at the discretion of the TAC Chair or who the TAC Chair designates.
When a maintainer has not had any activity in a particular project for three months, they will receive a notification informing them of the inactivity policies. The means and manner of notification (e.g., email, GitHub mentions) will be at the discretion of the TAC Chair or their designee.
Copy link
Contributor

Choose a reason for hiding this comment

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

I recommend that the "TAC chair" in this statement be changed to the "project's TSC chair"


A Hyperledger Foundation staff member will provide a report (or maintain an automated means to generate a report) of the most recent GitHub tracked actions for contributors at regular intervals to the TAC. It will be the TAC's responsibility to act on the data.
A LF Decentralized Trust staff member will provide a report (or maintain an automated means to generate a report) of the most recent GitHub tracked actions for contributors at regular intervals to the TAC. It will be the TAC's responsibility to act on the data.
Copy link
Contributor

Choose a reason for hiding this comment

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

Recommend changing "TAC" to "project's TSC"

Copy link
Member

@petermetz petermetz left a comment

Choose a reason for hiding this comment

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

LGTM in general, left some comments of support in existing suggestion threads.

inactivity.md Outdated Show resolved Hide resolved
inactivity.md Outdated Show resolved Hide resolved
index.md Outdated Show resolved Hide resolved
raising-an-issue.md Outdated Show resolved Hide resolved
Signed-off-by: S m, Aruna <arun.s.m.cse@gmail.com>
@arsulegai
Copy link
Member Author

@VRamakrishna . I cleaned up the working branch and removed stale and unrelated commits. The PR now has files related to maintainers inactivity policy.

@tkuhrt tkuhrt removed the quarterly-report Quarterly Project Updates label Sep 9, 2024
@tkuhrt tkuhrt closed this Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants