conjure

:neovim:+:clj: https://github.com/Olical/conjure - If you're enjoying Conjure and want to say thanks: https://github.com/users/Olical/sponsorship :smile: (alt https://conjure.fun/discord)
adambros 2020-08-12T19:18:59.005900Z

upgrading to 4.1.0 to 4.2.0 (wanted to turn on conjure#log#wrap) breaks my <localleader>cf (always gives "No nREPL port file found") copying .nrepl-port to src/dev when running ,cf in src/dev/user.clj fixes it seems like it's no longer doing this:

`g:conjure#client#clojure#nrepl#connection#port_files`
            ...
            Conjure checks every directory above your current one
            ...
I can make this a github issue if this is indeed a bug

Olical 2020-08-12T19:20:42.006400Z

Hmm that is most definitely still working fine for me.

Olical 2020-08-12T19:21:26.007400Z

What does :cwd show?

adambros 2020-08-12T19:21:40.007800Z

i have set autochdir so it shows src/dev

Olical 2020-08-12T19:22:25.008300Z

I've never used that option so maybe it's helping expose a more subtle issue

Olical 2020-08-12T19:22:39.008900Z

I'll take a look in ~30 minutes

👍 1
adambros 2020-08-12T19:22:49.009100Z

(technically i'm using au BufEnter * silent! lcd %:p:h, forget why lol)

adambros 2020-08-12T19:26:58.009500Z

This isn't urgent, I can work around it pretty easily for a day or two, so no rush from me

Olical 2020-08-12T19:47:39.010Z

I can repro! 😱 I'm sure it's something simple

Olical 2020-08-12T19:47:47.010300Z

Thank you for bringing it to my attention.

Olical 2020-08-12T19:50:03.010500Z

(the odd part is that the related code hasn't changed)

😱 1
adambros 2020-08-12T19:56:28.011Z

git bisect to the rescue?

Olical 2020-08-12T19:56:40.011200Z

Quite possibly :thinking_face:

Olical 2020-08-12T20:02:45.011700Z

Oh wow, I know what it is.

Olical 2020-08-12T20:02:49.011900Z

🤦

Olical 2020-08-12T20:03:03.012300Z

I've got two functions called resolve in my fs module

Olical 2020-08-12T20:03:40.012700Z

Which works similarly, but not the same, which is why it was hidden partially

Olical 2020-08-12T20:09:52.012900Z

This should be fixed on develop

Olical 2020-08-12T20:10:02.013200Z

If so, I'll get this published in a patch

Olical 2020-08-12T20:11:50.013700Z

@adambros (sorry I should've used a thread! Also no rush to test it!)

Olical 2020-08-12T20:12:27.014100Z

I have now renamed all of the functions that were waaaay to close in name to much more specific names.

Olical 2020-08-12T20:12:56.014500Z

Also thank you so much @orestis for the sponsorship! 😱 ❤️

Olical 2020-08-12T20:13:50.015100Z

DM me if you're interested in me posting you a couple of Conjure stickers, I'm building up a list of addresses, I'll make a post office run some day soon-ish

Olical 2020-08-12T20:13:58.015400Z

(although I'm away on holiday next week)

adambros 2020-08-12T20:26:00.016400Z

@olical can confirm that develop fixed it, thanks for the quick turnaround!

🎉 1
2020-08-12T20:30:38.016600Z

2020-08-12T20:31:17.016700Z

2020-08-12T20:31:28.016800Z

2020-08-12T20:31:52.016900Z

Olical 2020-08-12T20:33:01.017400Z

Fixed and released, although I accidentally tagged the wrong commit at first... so... released twice!