-
Notifications
You must be signed in to change notification settings - Fork 46
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
Updates to the Annual Report Process and Template for LFDT #289
Conversation
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
- revert changes on history to no break working links - add TODOs to mark changes that needs to be made once information is available Signed-off-by: Marcus Brandenburger <bur@zurich.ibm.com>
Signed-off-by: Marcus Brandenburger <bur@zurich.ibm.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Tracy A. Kuhrt <tracy.a.kuhrt@accenture.com>
Signed-off-by: Stephen Curran <swcurran@gmail.com>
Changes in the PR:
|
</mark> | ||
|
||
# Project Health | ||
<mark>_Include a link to your project’s [LFX Insights page](https://insights-v2.lfx.linuxfoundation.org/projects). We will be looking for signs of consistent or increasing contribution activity. Please feel free to add commentary to add color to the numbers and graphs we will see on Insights._ | ||
## Project Health |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] I’m curious why the entire file appears as different. It seems like an extra # was added for the headers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was cleanup of the headers — fixing Markdown Lint warnings. The front matter puts in the “#” header, so the rest should be “##” headers.
- Agreeing between themselves who will complete the annual review. | ||
- Informing the community about the annual review and requesting input. | ||
- Creating the annual review by filing a PR in GitHub under [lf-decentralized-trust/toc/project-reports/](https://github.com/lf-decentralized-trust/toc/tree/gh-pages/project-reports). | ||
- Sending email to the [TAC mailing list](mailto:toc@lists.lf-decentralized-trust.org) and to the project community so that the community knows the PR is there and can comment on it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: Is the mailing list a TODO?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely need a toc
to tac
, but yes, it is a ToDo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just made the tac
to toc
change, but I think a link checker can find this one if it is wrong.
|
||
- Agreeing between themselves who will complete the annual review. | ||
- Informing the community about the annual review and requesting input. | ||
- Creating the annual review by filing a PR in GitHub under [lf-decentralized-trust/toc/project-reports/](https://github.com/lf-decentralized-trust/toc/tree/gh-pages/project-reports). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: Is the github link a TODO?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I just need to change toc
to tac
in it. But a link check should be run on all of these docs after publication to make sure they are right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed the toc
to tac
where appropriate. There is one instance that was purposely left.
|
||
### TAC Secondary Member | ||
- Working closely with the lead TAC member to review and corroborate their findings | ||
|
||
- Working closely with the lead TAC member to review thew project's annual report, and corroborate and publish their findings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Working closely with the lead TAC member to review thew project's annual report, and corroborate and publish their findings. | |
- Working closely with the lead TAC member to review thew project's annual report, and to corroborate and publish their findings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
* A majority of the TAC members agree to continue the project at its current stage, or | ||
* A majority of the TAC members recommend that a project be moved to a new stage, including `End of Life`, or | ||
* If the TAC cannot come to a consensus, the project will remain at its current stage. | ||
- A majority of the TAC members agree to continue the project at its current status, or |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: Is the term project's status
broader than project's stage
or do they carry the same meaning? If status
is broader, does it imply an inferred condition based on the project's report and current stage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per the discussion at the TOC call on 20240829 — the term “status” and “stage” were used seemingly interchangeably in the document. I didn’t think that was clear, so I checked the “lifecycle” document and it uses the term “status”, so I changed all “stage” references to “status”.
- remove the `nav_exclude` line | ||
- Text between `<mark></mark>` are instructions. Please remove when section has been completed. | ||
* Send an email to the [TAC mailing list](mailto:toc@lists.hyperledger.org) so that the community knows the PR is there and can comment on it. | ||
Project TSC members are responsible for agreeing between them who will complete the annual review. One of the TSC members should create the review in GitHub under [lf-decentralized-trust/toc/project-reports/](https://github.com/lf-decentralized-trust/toc/tree/gh-pages/project-reports). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project TSC members are responsible for agreeing between them who will complete the annual review. One of the TSC members should create the review in GitHub under [lf-decentralized-trust/toc/project-reports/](https://github.com/lf-decentralized-trust/toc/tree/gh-pages/project-reports). | |
Project TSC members are responsible for agreeing between them who will complete the annual review. One of the project's TSC members should create the review in GitHub under [lf-decentralized-trust/toc/project-reports/](https://github.com/lf-decentralized-trust/toc/tree/gh-pages/project-reports). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
- remove the `nav_exclude` line | ||
- Text between `<mark></mark>` are instructions. Please remove when section has been completed. | ||
* Send an email to the [TAC mailing list](mailto:toc@lists.hyperledger.org) so that the community knows the PR is there and can comment on it. | ||
Project TSC members are responsible for agreeing between them who will complete the annual review. One of the TSC members should create the review in GitHub under [lf-decentralized-trust/toc/project-reports/](https://github.com/lf-decentralized-trust/toc/tree/gh-pages/project-reports). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: Should the github link be a TODO?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The links can be checked by a link checker when the new repo is used. I don’t think we need a todo on each where we probably have it right. I did put a couple of “TBDs” where I didn’t know at all what the link would be.
* A TAC representative will be chosen on a round-robin basis to lead the review once the project files a PR. This representative should be someone that is not directly involved with the project. | ||
* A secondary representative will be chosen on a round-robin basis to observe and corroborate the findings of the lead. This representative should be someone that is not directly involved with the project. | ||
* The lead TAC member reviews the content of the PR and analyzes the project for community health indicators, their findings are placed within a thread in the TAC channel ([#toc](https://discord.com/channels/905194001349627914/941384040316018790)) for discussion. The thread should contain: | ||
1. important facts about the project that could influence the TACs decision around the future of the project, its current stage, and path to other stages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I notice that these sentences have been removed. It seems necessary to establish guidance that suggests the review should be objective and holistic, considering all aspects such as encouraging projects to plan for the next stage in their lifecycle, as well as ensuring ongoing viability.
|
||
**NOTE:** If a project has genuinely stalled we can save everyone’s time and effort by archiving the project and moving it to `End of Life`. | ||
Annual reviews will replace the Q1 quarterly report. **If a project's annual review is not submitted within two months of notification, we will take this as a sign that the project is not under active maintenance and the TAC is likely to decide to archive the project and move it to End of Life status.** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Annual reviews will replace the Q1 quarterly report. **If a project's annual review is not submitted within two months of notification, we will take this as a sign that the project is not under active maintenance and the TAC is likely to decide to archive the project and move it to End of Life status.** | |
Annual reviews will replace the Q1 quarterly report. **If a project's annual review is not submitted within two months of notification, the TAC will take this as a sign that the project is not under active maintenance and is likely to decide to archive the project and move it to End of Life status.** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
**NOTE:** If a project has genuinely stalled we can save everyone’s time and effort by archiving the project and moving it to `End of Life`. | ||
Annual reviews will replace the Q1 quarterly report. **If a project's annual review is not submitted within two months of notification, we will take this as a sign that the project is not under active maintenance and the TAC is likely to decide to archive the project and move it to End of Life status.** | ||
|
||
**NOTE:** If a project has genuinely stalled the project's TSC can save everyone’s time and effort by archiving the project and moving it to `End of Life`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: Would project's TSC decide to EOL their project or do they bring it to LFDT TAC for the move?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the TSC has the governance authority to do it. The mechanics would be situational. This sentence is a cleanup / clarification (at least it clarified it for me) on what was in the document already.
I obviously have way too many commits in this — branched from the wrong fork. Will close this, open a new PR and reference this PR to capture the review comments made. I’ve addressed all of the comments from the review. |
Signed-off-by: Stephen Curran swcurran@gmail.com