-
Notifications
You must be signed in to change notification settings - Fork 181
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: run prettier on generated output #343
Conversation
Codecov Report
@@ Coverage Diff @@
## main #343 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 21 21
Lines 633 633
Branches 37 37
=========================================
Hits 633 633 Continue to review full report at Codecov.
|
@IvanGoncharov We don't even publish this as a branch on GitHub so wouldn't make much difference if we prettify output? |
package-lock.json
Outdated
@@ -6999,6 +6999,7 @@ | |||
"version": "3.4.0", | |||
"resolved": "https://registry.npmjs.org/uuid/-/uuid-3.4.0.tgz", | |||
"integrity": "sha512-HjSDRw6gZE5JMggctHBcjVak08+KEVhSIiDzFnT9S9aegmp85S/bReBVTb4QTFaRNptJ9kuYaNhnbNEOkbKb/A==", | |||
"deprecated": "Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not critical.
But in the future, can you please checklist of changed files to see unrelated files?
As requested in https://github.com/graphql/graphql-relay-js/pull/341/files#r655170705