You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I just noticed this on the #clojure channel on Slack; seems like detecting e.g. (ns a–b), which has an em dash, as problematic (versus (ns a-b), which uses a normal dash) might be useful:
noisesmith Feb 22nd at 2:39 PM
I fat fingered with compose-keys turned on, and accidentally discovered that if the ns name in the ns form has an em-dash instead of a dash, require returns without error but it creates a broken namespace. of course the answer is "don't use em-dash in the namespace symbol", but I wonder if there's a linter that checks for things like that
2 replies
andy.fingerhut 2 days ago
borkdude enhancing clj-kondo in 5... 4... 3... 2... 🙂
andy.fingerhut 2 days ago
Certainly sounds like a linter that checks for strict ASCII subset (for some fraction of developers), or a larger set that includes other alphabets could find this quickly.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
I just noticed this on the
#clojure
channel on Slack; seems like detecting e.g.(ns a–b)
, which has an em dash, as problematic (versus(ns a-b)
, which uses a normal dash) might be useful:noisesmith Feb 22nd at 2:39 PM
I fat fingered with compose-keys turned on, and accidentally discovered that if the ns name in the ns form has an em-dash instead of a dash, require returns without error but it creates a broken namespace. of course the answer is "don't use em-dash in the namespace symbol", but I wonder if there's a linter that checks for things like that
2 replies
andy.fingerhut 2 days ago
borkdude enhancing clj-kondo in 5... 4... 3... 2... 🙂
andy.fingerhut 2 days ago
Certainly sounds like a linter that checks for strict ASCII subset (for some fraction of developers), or a larger set that includes other alphabets could find this quickly.
Beta Was this translation helpful? Give feedback.
All reactions