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

Update funding process with 2025 schedule #435

Merged
merged 4 commits into from
Jan 15, 2025
Merged

Conversation

steiza
Copy link
Member

@steiza steiza commented Jan 13, 2025

For #419

There wasn't consensus on rolling evaluation vs batching, so I decided on batching every 2 months instead of every 3 months. This gives us 5 review periods (without trying to squeeze one in at the end of the year). Review periods by the TAC are two weeks long, in case some TAC members are on vacation or otherwise temporarily unavailable to vote.

This change also streamline workflow / review process and addd examples of successful funding requests.

It does not change the funding process from using issues to using pull requests, although I'm still considering that for a follow-up change. For the upcoming review cycle we already have at least 1 issue to include in our review anyways, namely #424.

- Streamline workflow / review process
- Add examples of successful funding requests

Signed-off-by: Zach Steindler <steiza@github.com>
@steiza steiza requested a review from a team as a code owner January 13, 2025 20:12
Signed-off-by: Zach Steindler <steiza@github.com>
Copy link
Contributor

@mlieberman85 mlieberman85 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@scovetta scovetta left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Zach Steindler <steiza@github.com>
Copy link
Contributor

@marcelamelara marcelamelara left a comment

Choose a reason for hiding this comment

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

Thanks for these updates @steiza ! I only have one minor comment that isn't a blocker. I'm also in favor of following this up with a transition from issues to PRs for funding requests :)

Copy link
Contributor

@lehors lehors left a comment

Choose a reason for hiding this comment

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

LGTM but I'm all for PRs over Issues.

Signed-off-by: Zach Steindler <steiza@github.com>
@steiza
Copy link
Member Author

steiza commented Jan 15, 2025

I think this falls under "Content" on the TAC decision process in which case it has the necessary approvals - thanks everyone!

@steiza steiza merged commit 2f799f1 into main Jan 15, 2025
1 check passed
@steiza steiza deleted the 2025-funding-process branch January 15, 2025 15:30
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