Graphql-js: Error with flowlint in [email protected]

Created on 23 Aug 2019  ยท  6Comments  ยท  Source: graphql/graphql-js

I got the error with flow in [email protected]. I do not get any error in [email protected]. Can someone help me? Or maybe you can remove this rule of flowlint?

$ /Users/hsuting/Desktop/work/core/node_modules/.bin/flow
Error โ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆ node_modules/graphql/type/definition.js.flow:4:13

Redundant argument. This argument doesn't change any lint settings.

     1โ”‚ // @flow strict
     2โ”‚
     3โ”‚ // FIXME
     4โ”‚ // flowlint deprecated-type: off
     5โ”‚ import objectEntries from '../polyfills/objectEntries';
     6โ”‚
     7โ”‚ import inspect from '../jsutils/inspect';
bug

Most helpful comment

@HsuTing @gajus I'm working on 14.5.1 to fix this issue ASAP. ETA < 1hr.

All 6 comments

Is there a way to suppress this error?

@HsuTing @gajus I'm working on 14.5.1 to fix this issue ASAP. ETA < 1hr.

@HsuTing @gajus Sorry for the delay.
Had unexpected meeting :(
Fixed in 14.5.1 ๐Ÿ“ฆ

@IvanGoncharov
That is ok. Thank you. ๐Ÿ‘

@IvanGoncharov Looks like this has come back?

$ yarn flow status
yarn run v1.13.0                                                                                                                                                        
warning package.json: No license field                                                                                                                                  
$ /nail/home/markl/pg/dlcg/examples/swapi/node_modules/.bin/flow status                                                                                                 
Launching Flow server for /nail/home/markl/pg/dlcg/examples/swapi
Spawned flow server (pid=22550)
Logs will go to /tmp/flow/zSnailzShomezSmarklzSpgzSdlcgzSexampleszSswapi.log
Monitor logs will go to /tmp/flow/zSnailzShomezSmarklzSpgzSdlcgzSexampleszSswapi.monitor_log
Error โ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆโ”ˆ node_modules/graphql/error/GraphQLError.js.flow:4:13

Redundant argument. This argument doesn't change any lint settings.

     1โ”‚ // @flow strict
     2โ”‚
     3โ”‚ // FIXME:
     4โ”‚ // flowlint uninitialized-instance-property:off
     5โ”‚
     6โ”‚ import isObjectLike from '../jsutils/isObjectLike';
     7โ”‚ import { SYMBOL_TO_STRING_TAG } from '../polyfills/symbols';



Found 1 error
error Command failed with exit code 2.                                                                                                                                  
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.                                                                                    
$ yarn list graphql
yarn list v1.13.0                                                                                                                                                       
warning package.json: No license field                                                                                                                                  
warning No license field                                                                                                                                                
warning Filtering by arguments is deprecated. Please use the pattern option instead.                                                                                    
โ””โ”€ [email protected]
Done in 0.25s.                                                                                                                                                          

$ yarn list flow-bin
yarn list v1.13.0                                                                                                                                                       
warning package.json: No license field                                                                                                                                  
warning No license field                                                                                                                                                
warning Filtering by arguments is deprecated. Please use the pattern option instead.                                                                                    
โ””โ”€ [email protected]
Done in 0.23s.       

This has indeed comeback.

Your best bet to workaround it is to mark it as untyped:

[untyped]
<PROJECT_ROOT>/node_modules/graphql/error/GraphQLError.js.flow

Was this page helpful?
0 / 5 - 0 ratings

Related issues

adriano-di-giovanni picture adriano-di-giovanni  ยท  3Comments

swist picture swist  ยท  4Comments

scf4 picture scf4  ยท  3Comments

ablbol picture ablbol  ยท  4Comments

gjuchault picture gjuchault  ยท  4Comments