Olical / conjure

Interactive evaluation for Neovim (Clojure, Fennel, Janet, Racket, Hy, MIT Scheme, Guile, Python and more!)
https://conjure.oli.me.uk
The Unlicense
1.79k stars 110 forks source link

ConjureSchool works on Linux, breaks on Windows 10 #384

Open sirwobin opened 2 years ago

sirwobin commented 2 years ago

On windows I start the tutorial with :ConjureSchool, \eb lesson 1 (works fine), \ee inside commented form lesson 2 (works fine), \er inside the do form and the log buffer indicates a nil return value and the conjure-school.fnl buffer does not change. "Hello world" is not printed.

On ubuntu (running under WSL on the same laptop) the third lesson appears to work correctly, adding to conjure-school.fnl buffer, printing "Hello world" below in the screen and adding hello world output to the log buffer.

Windows 10 running under PowerShell NVIM v0.7.2 Build type: RelWithDebInfo LuaJIT 2.1.0-beta3 Compiled by runneradmin@fv-az276-503 Features: -acl +iconv +tui

Ubuntu 20.04 running under zsh NVIM v0.7.2 Build type: Release LuaJIT 2.1.0-beta3 Compiled by team+vim@tracker.debian.org Features: +acl +iconv +tui

Olical commented 2 years ago

Hmm were you using tree sitter by any chance? And if so... that may be fixed now! If not, I'm afraid I still have nothing to offer since I'm not sure how to install Neovim + Conjure on Windows 😞

paotsaq commented 9 months ago

@Olical hello! thanks for this work – I'm really excited to try it out.

I was trying to do ConjureSchool, but the mapping wasn't working.

I had this on my conjure-school.fnl (both when doing :ConjureSchool and the curl on the readme.

;; Let's begin by evaluating the whole buffer using <localleader>eb
;; Your <localleader> wasn't configured so I've defaulted it to comma (,) for now.
;; See :help localleader for more information. (let maplocalleader=",")
(school.lesson-1)

However, @sirwobin above had \ as the local leader — and so do I! Is this supposed to happen?

thank you! :sparkles:

Olical commented 9 months ago

So the defaulting to \ was a bug! I've fixed that so it properly defaults to , now, it was setting it to , but due to a timing related thing the mappings were defined with a \ and couldn't be updated after the maplocalleader setting had been updated. I've changed the order of some calls around so this is no longer a problem.