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

docs: reviewer and merge process #4013

Merged
merged 3 commits into from
Jan 24, 2024
Merged

docs: reviewer and merge process #4013

merged 3 commits into from
Jan 24, 2024

Conversation

saihaj
Copy link
Member

@saihaj saihaj commented Jan 24, 2024

In GraphQL.js working group we discussed and put process in place. But we never officially wrote down in docs. You can watch this recording https://youtu.be/BBxc_HmB71I?si=cvdUMI47vBsT9F2y&t=1316 or check the transcript. These are the changes we agreed in the working group, just putting them in writing.

Copy link

netlify bot commented Jan 24, 2024

Deploy Preview for compassionate-pike-271cb3 ready!

Name Link
🔨 Latest commit e80d809
🔍 Latest deploy log https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/65b14707adc8ff0009cfb3ba
😎 Deploy Preview https://deploy-preview-4013--compassionate-pike-271cb3.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

Hi @saihaj, I'm @github-actions bot happy to help you with this PR 👋

Supported commands

Please post this commands in separate comments and only one per comment:

  • @github-actions run-benchmark - Run benchmark comparing base and merge commits for this PR
  • @github-actions publish-pr-on-npm - Build package from this PR and publish it on NPM

Copy link
Member

@IvanGoncharov IvanGoncharov left a comment

Choose a reason for hiding this comment

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

@saihaj Thanks for raising this PR.

At the moment, I don't have capacity for this project.
So it would be great if others help.

Copy link
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

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

Should the handle be @graphql/graphql-js-reviewers?

Copy link
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

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

LGTM; I've added a suggestion but only add it if this is something you've already discussed (I seem to recall this is something that was mentioned).

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
Co-authored-by: Benjie <benjie@jemjie.com>
@saihaj saihaj merged commit 9c90a23 into main Jan 24, 2024
34 checks passed
@saihaj saihaj deleted the saihaj/reviewer-process branch January 24, 2024 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants