Skip to content

Conversation

IvanGoncharov
Copy link
Member

Motivation: I'm working on the similar code in parser and don't want to
replicate ESLint disable comment so figured out a proper way to solve
it. Also throwing error with message will help with debuging if someone
will have an issue with validate code.

@IvanGoncharov IvanGoncharov added the PR: polish 💅 PR doesn't change public API or any observed behaviour label Jul 22, 2022
@netlify
Copy link

netlify bot commented Jul 22, 2022

Deploy Preview for compassionate-pike-271cb3 ready!

Name Link
🔨 Latest commit ce9bd63
🔍 Latest deploy log https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/62dfe88d46b48b00086c3faf
😎 Deploy Preview https://deploy-preview-3680--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 settings.

@github-actions
Copy link

Hi @IvanGoncharov, 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

Motivation: I'm working on the similar code in parser and don't want to
replicate ESLint disable comment so figured out a proper way to solve
it. Also throwing error with message will help with debuging if someone
will have an issue with `validate` code.
@IvanGoncharov IvanGoncharov merged commit 5aadd61 into graphql:main Jul 26, 2022
@IvanGoncharov IvanGoncharov deleted the pr_branch4 branch July 26, 2022 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: polish 💅 PR doesn't change public API or any observed behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant