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

feat(doc): Update maintainer list according to activity and add steering commitee member… #3311

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

leonrayang
Copy link
Member

@leonrayang leonrayang commented Apr 12, 2024

…ring commitee member

Motivation

Explain here the context, and why you're making that change. What is the problem you're trying to solve.


Some maintainers are not active enough, or there are too many overlapping positions within the maintainers.
A steering committee has been added to better achieve community autonomy governance

For a long time, important project decisions have been made by core maintainers through platforms like WeChat. In order to improve project governance and ensure transparency of information, a Steering Committee has been established. The initial members of the Steering Committee are selected from the current core maintainers. Following the principle of vendor neutrality, the committee consists of core members from several companies that have the highest level of contribution and influence in the project.

Modifications

Describe the modifications you've done.


@bboyCH4 @jadewang198510 transfer to committer

Types of changes

Show in a checkbox-style, the expected types of changes your project is supposed to have:


  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Bugfix (non-breaking change which fixes an issue)
  • Documentation Update (if none of the other choices apply)
  • So on...

Verifying this change

(Please pick either of the following options.)


  • Make sure that the change passes the testing checks.

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • This can be verified in development debugging
  • This can be realized in a mocked environment, like a test cluster consisting in docker

(or)

This change MUST reappear in online clusters, or occur in that specific scenarios.

Does this pull request potentially affect one of the following parts:

Which of the following parts are affected by this change?


  • Master
  • MetaNode
  • DataNode
  • ObjectNode
  • AuthNode
  • LcNode
  • Blobstore
  • Client
  • Cli
  • SDK
  • Other Tools
  • Common Packages
  • Dependencies
  • Anything that affects deployment

Documentation

Is there a chinese and english document modification?


  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Review Expection

How long would you like the team to be completed in your contributing?


  • in-two-days
  • weeks
  • free-time
  • whenever

Matching PR in forked repository

enter the url if has PR in forked repository.


PR in forked repository:

…ing commitee member

Signed-off-by: leonrayang <changliang@oppo.com>
Copy link

netlify bot commented Apr 12, 2024

Deploy Preview for cubefs-check ready!

Name Link
🔨 Latest commit 450fa70
🔍 Latest deploy log https://app.netlify.com/sites/cubefs-check/deploys/66189bf07aba5a0009d2dc99
😎 Deploy Preview https://deploy-preview-3311--cubefs-check.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@mervinkid mervinkid left a comment

Choose a reason for hiding this comment

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

Approve to merge.

@awzhgw awzhgw requested review from awzhgw and removed request for guowl0919 April 12, 2024 03:18
Copy link
Contributor

@xiaochunhe xiaochunhe left a comment

Choose a reason for hiding this comment

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

LGTM

@leonrayang leonrayang merged commit 6f723d7 into cubefs:master Apr 12, 2024
17 checks passed
@leonrayang
Copy link
Member Author

+1

5 similar comments
@Cloudstriff
Copy link
Member

+1

@Victor1319
Copy link
Member

+1

@sejust
Copy link
Collaborator

sejust commented Apr 18, 2024

+1

@cessory
Copy link
Collaborator

cessory commented Apr 18, 2024

+1

@bboyCH4
Copy link
Contributor

bboyCH4 commented Apr 18, 2024

+1

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.

None yet

9 participants