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

A script to manage members of the org #114

Closed
wants to merge 1 commit into from
Closed

A script to manage members of the org #114

wants to merge 1 commit into from

Conversation

williamkapke
Copy link
Contributor

I have reworked the original POC code from #29 and made it ready to deploy.

/cc @nodejs/tsc

How it works

Modify a repo's README.md

Place a special <!-- team:name --> comment in the repo's readme.md:

## Members
<!-- team:website - all mentions in this section will be added to the team by the bot -->
- @williamkapke
- @Fishrock123

<!-- team -->

NOTE: You must specify the team's name in the opening comment like shown above!

Create/update a PR

When a PR is submitted or changes are pushed, the bot will evaluate the commit(s) for changes to the README.md file. If changed, it will extract the content within the <!-- team --> comments find all @mentions and do a diff with the list of current team members.

Accept the PR

Accepting a PR sends a push event. So, any push- even ones outside the PR process, will get picked up by the bot.

When the bot receives the push event, it will do the same parsing + evaluating as when the PR was created/updated, but instead of creating a comment- it applies the additions AND removals to the team (if any).

Other changes in this PR

A few of the integration tests were changed to limit them to loading only the script it tests (via process.env.SCRIPTS). This prevents initialization code in unrelated scripts from running multiple times when running npm test.

@williamkapke
Copy link
Contributor Author

I'll leave this open a few more days incase someone is interested.

gessicagba

This comment was marked as off-topic.

phillipj

This comment was marked as off-topic.

phillipj

This comment was marked as off-topic.

@outsideris
Copy link

Any update for this?

@williamkapke
Copy link
Contributor Author

@outsideris As you saw- I've moved this effort to https://github.com/williamkapke/orgbot

Now I need to get it in to production. Thanks for you patience!

@outsideris
Copy link

@williamkapke Awesome!!!!!
Where can I track the production state for node in?

@gessicagba
Copy link

Hi

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.

4 participants