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

[Needs SC vote before merging] Steering Committee: team membership #1237

Merged
merged 4 commits into from Apr 13, 2024

Conversation

Andersson007
Copy link
Contributor

See https://forum.ansible.com/t/adding-a-core-team-member-to-the-steering-commitee/4492/6

Let's keep it as simple as possible not optimizing prematurely.

@Andersson007 Andersson007 requested a review from a team as a code owner March 26, 2024 08:56
@Andersson007 Andersson007 marked this pull request as draft March 26, 2024 08:56
@ansible-documentation-bot ansible-documentation-bot bot added the sc_approval This PR requires approval from the Ansible Community Steering Committee label Mar 26, 2024
@@ -46,6 +46,8 @@ The following table lists the current Steering Committee members. See :ref:`stee
+------------------+---------------+-------------+
| Brian Scholer | briantist | 2022 |
+------------------+---------------+-------------+
| Core Team repr. | See Forum | 2024 |
Copy link
Contributor

Choose a reason for hiding this comment

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

If the Community Topic in the forum exists, I think we should have a link to it here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It doesn't exist yet but how about just adding a link afterwards w/o a vote? We first need to vote on this PR imo as the core-team representative announcement topic depends on approval of this PR by SC,
If you want to create it now it wouldn't hurt either, so please go ahead and paste the link here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, I think I've put this badly. I should have said when the Community Topic in the forum exists.

So vote on this, merge your PR (if the vote is positive) and add the link afterwards.

@Andersson007
Copy link
Contributor Author

Folks, i'm gonna be on PTO between March 28 and April 8. Please don't wait for me.
@samccann or whoever has permissions can commit good suggestions.
also anyone from the SC can start a vote at any time.
Please don't wait and go ahead. Happy with any changes the community reaches an agreement on:)

Co-authored-by: Sandra McCann <samccann@redhat.com>
Copy link
Contributor

@mariolenz mariolenz left a comment

Choose a reason for hiding this comment

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

LGTM

@mariolenz
Copy link
Contributor

The vote is over: The proposal / this PR has been accepted.

Can someone please un-draft and merge?

@felixfontein felixfontein marked this pull request as ready for review April 13, 2024 11:31
@felixfontein felixfontein merged commit 815d52b into ansible:devel Apr 13, 2024
8 checks passed
@felixfontein
Copy link
Collaborator

Thanks everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sc_approval This PR requires approval from the Ansible Community Steering Committee
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants