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

Build a script that grants TSC members to release @wdio packages #4737

Merged
merged 1 commit into from
Nov 6, 2019

Conversation

christian-bromann
Copy link
Member

There are a lot of @wdio packages that need to be released. As per our governance only TSC members are allowed to release packages. To make on-boarding of new members easier and more automatic, let's build a script that calls NPMs API and registers members (maintained in an extra json file in the repository).

Also add a sentence in the Governance.md to run the script as part of the on-boarding.

@christian-bromann christian-bromann self-assigned this Nov 5, 2019
@christian-bromann
Copy link
Member Author

We should also add a comment that helps TSC members to better reason about when to make a release and how to apply the SemVer concept to the monorepo.

@erwinheitzman
Copy link
Member

We should also add a comment that helps TSC members to better reason about when to make a release and how to apply the SemVer concept to the monorepo.

I would definitely appreciate this 👍

@christian-bromann
Copy link
Member Author

It turns out that NPM doesn't really have an API client and that they prefer this via their web interface. Therefor I added a section to the Governance description to add the user to the org which automatically grants publish rights.

@christian-bromann christian-bromann requested a review from a team November 6, 2019 19:18
@christian-bromann christian-bromann merged commit ca55515 into master Nov 6, 2019
@christian-bromann christian-bromann deleted the cb-update-release-guide branch November 6, 2019 19:34
@christian-bromann christian-bromann added the PR: Internal 🏠 PRs that contain changes internals label Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Internal 🏠 PRs that contain changes internals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants