Skip to content

Commit

Permalink
Merge pull request #22 from kepler16/symbol-resolution-fix
Browse files Browse the repository at this point in the history
fix(resolve-symbol): pass through non symbols
  • Loading branch information
armed authored Jan 10, 2023
2 parents 7ee0fac + 9d051a2 commit fdd8103
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/k16/gx/beta/core.cljc
Original file line number Diff line number Diff line change
Expand Up @@ -326,8 +326,8 @@
(with-err-ctx err-ctx
(if (fn? resolved-props-fn)
(run-props-fn resolved-props-fn dep-nodes-vals)
(impl/postwalk-evaluate
dep-nodes-vals resolved-props props)))
(impl/postwalk-evaluate
dep-nodes-vals resolved-props props)))
validate-error (with-err-ctx err-ctx
(validate-props props-schema props-result))
[error result] (when-not validate-error
Expand Down
5 changes: 3 additions & 2 deletions src/k16/gx/beta/impl.cljc
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@

(defn resolve-symbol
[sym]
(when (symbol? sym)
(if (symbol? sym)
(if-let [nss #?(:cljs (namespace-symbol sym)
:clj (try
(some->> sym
Expand All @@ -195,7 +195,8 @@
:exception e}))))]
nss
(gx.err/add-err-cause {:title :symbol-cannot-be-resolved
:data sym}))))
:data sym}))
sym))

(defn form->runnable [form-def]
(let [props* (atom #{})
Expand Down

1 comment on commit fdd8103

@vercel
Copy link

@vercel vercel bot commented on fdd8103 Jan 10, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

gx – ./

gx.kepler16.com
gx-git-master-kepler16.vercel.app
gx-kepler16.vercel.app

Please sign in to comment.