Skip to content
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

clj-kondo chokes on string namespaces in require #51

Closed
borkdude opened this issue Apr 5, 2019 · 0 comments
Closed

clj-kondo chokes on string namespaces in require #51

borkdude opened this issue Apr 5, 2019 · 0 comments
Labels
bug Something isn't working

Comments

@borkdude
Copy link
Member

borkdude commented Apr 5, 2019

CLJS projects can have string namespaces. E.g.:

(ns myapp
  (:require [reagent.core :as r :refer [atom]]
            ["react-native" :as rn :refer [AppRegistry]]
            ["react-native-navigation" :as rnn]
            [env]))

Currently clj-kondo chokes on processing theses file. Moreover, it doesn't process the rest of the directory when one of the files is not readable. Both issues should be fixed.

borkdude added a commit that referenced this issue Apr 5, 2019
borkdude added a commit that referenced this issue Apr 5, 2019
borkdude added a commit that referenced this issue Apr 5, 2019
borkdude added a commit that referenced this issue Apr 5, 2019
@borkdude borkdude added the bug Something isn't working label Apr 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant