search

retro / graphql-builder

GraphQL client library for Clojure and ClojureScript
MIT License
184 stars 15 forks source link

Use of undeclared Var after upgrade to 1.7 #12

Closed kurt-o-sys closed 6 years ago

kurt-o-sys commented 6 years ago

After upgrading to 1.7, I get errors using graphql-builder (all works fine in 1.6):

(ns ...
  (:require [graphql-builder.core :as core]
            [graphql-builder.parser :refer-macros [defgraphql]]
            ...]))
...
(defgraphql graphql-queries
  "src/graphql/file1.graphql"
  "src/graphql/file2.graphql"
  ...)

results in:

[:app] Build completed. (1229 files, 1068 compiled, 2 warnings, 47.83s)

------ WARNING #1 --------------------------------------------------------------
 File: /...:21:1
--------------------------------------------------------------------------------
  18 |                                           ...)))
  19 |     ...))
  20 | 
  21 | (defgraphql graphql-queries
-------^------------------------------------------------------------------------
 Use of undeclared Var graphql-builder.parser/parse
--------------------------------------------------------------------------------
  22 |   "src/graphql/.file1graphql"
  23 |   "src/graphql/file2.graphql"
  24 |   ...
  25 |   ...)
--------------------------------------------------------------------------------

------ WARNING #2 --------------------------------------------------------------
 File: /...:21:1
--------------------------------------------------------------------------------
  18 |                                           ...)))
  19 |     ...))
  20 | 
  21 | (defgraphql graphql-queries
-------^------------------------------------------------------------------------
 Use of undeclared Var graphql-builder.parser/read-file
--------------------------------------------------------------------------------
  22 |   "src/graphql/file1.graphql"
  23 |   "src/graphql/file2.graphql"
  24 |   ...
  25 |   ...)
--------------------------------------------------------------------------------
retro commented 6 years ago

@kurt-o-sys Are you using it from Clojure or from ClojureScript?

kurt-o-sys commented 6 years ago

clojurescript...

retro commented 6 years ago

@kurt-o-sys Thank you for reporting this. New version (0.1.8) is pushed which fixes this bug

kurt-o-sys commented 6 years ago

nice, thx....

Still weird why gensym (parsed#) doesn't work on cljs. It should work as well, or am I missing something?