cursive

Issues at: https://git.io/cursive-issues
cfleming 2020-12-04T01:58:39.309700Z

https://twitter.com/CursiveIDE/status/1334678405002842112

10šŸŽ‰
kenny 2020-12-04T15:26:55.328500Z

I see a new play button next to my deftests. If I click it I see several options: Run test, debug test, run with cpu profiler, ... If I click run with cpu profiler, I get a red popup in the corner pointing to the Run tab saying "Error running 'Test user/my-test': Internal error: Don't know how to patch selected configuration" Is running with cpu profiler not currently supported?

cfleming 2020-12-04T19:47:55.330Z

Iā€™m not sure about that actually, Iā€™ll check it. Iā€™m hoping to add coverage support, but itā€™s more complicated than I hoped because of how Clojure works.

cfleming 2020-12-05T01:44:47.332900Z

Actually @abdusalam sorry, I can repro your error, I was trying it with a local REPL not a remote one. Iā€™ll fix that.

1šŸ‘
cfleming 2020-12-05T02:08:53.333200Z

Also fixed the indentation, and the marking elements disabled is now more consistent and obvious too.

1ā¤ļø1šŸŽ‰
salam 2020-12-05T05:32:01.334200Z

i just checked out the new EAP build and it looks great! now that the tabs have a close button (X) in them, is there any difference between that button and the one in the toolbar of the REPL tool window? the latter seems redundant now?

cfleming 2020-12-05T07:54:57.334600Z

Possibly, I think mostly it makes that consistent with the run config action toolbar, although youā€™re right that itā€™s probably not needed.

ikitommi 2020-12-05T19:27:53.338100Z

copy to scratch file šŸ‘ šŸ‘ šŸ‘

AJ Jaro 2020-12-05T19:46:14.338400Z

Thanks @cfleming for fixing the namespaced keywords!! Huge!!

cfleming 2020-12-05T19:51:07.338600Z

Thanks everyone! @ajarosinski In recent IntelliJ versions that is becoming a complex and subtle area of the code which is also difficult to test automatically - let me know if you see any more weirdness.

AJ Jaro 2020-12-05T19:52:17.338800Z

For sure I definitely will do. I know how much I appreciate this tool so anyway I can help is great!

kenny 2020-12-05T22:29:59.339Z

@cfleming Not sure if this is the display name bug you mentioned. I updated to 1.10.0-eap2-2020.3 and all my run configs have this red X next to them.

cfleming 2020-12-06T02:13:07.339400Z

@kenny Did you get that with eap1?

kenny 2020-12-06T02:13:23.339700Z

Yes

cfleming 2020-12-06T02:14:17.339900Z

So thatā€™s a different problem that I talked about in the release notes - the migration which fixes the display name field for existing run configs doesnā€™t fix IDE-local ones, only project-shared ones.

cfleming 2020-12-06T02:14:49.340100Z

If you choose edit configurations and open those configs and then apply, that should fix it - the defaulting will be done in the editor.

kenny 2020-12-06T02:15:46.340700Z

Whatā€™s the difference between IDE-local and project-shared?

cfleming 2020-12-06T02:52:25.340900Z

If you look in the REPL run config, thereā€™s a checkbox in the top right - ā€œStore as project fileā€. If thatā€™s checked, then the run config will be stored as a separate file in .idea and can be shared in VCS etc.

kenny 2020-12-06T02:55:58.341400Z

Ah, got it. Will try this tomorrow. Thanks.

salam 2020-12-04T03:25:31.310300Z

i just tried the new test runner and it works like a charm! this is huge, especially for folks who are used to the traditional tdd approach in intellij idea. i just found a minor bug as shown in the screenshot attached. anyway, thanks a lot!

cfleming 2020-12-04T03:47:48.310700Z

Thanks! Iā€™m glad itā€™s useful, I already use it all the time. With that screenshot, did the error message persist even after entering a display name, or had that been defaulted when you opened it?

salam 2020-12-04T03:59:47.315900Z

yes and yes: when i created the run configuration, the error message was already there even though the Display Name was already pre-populated with the default value that was the same as that for Name (ā€œUnnamedā€). changing the Display Name didnā€™t make the error message go away.

cfleming 2020-12-04T04:00:08.316100Z

Hmm, strange, thanks - Iā€™ll try to reproduce that.

cfleming 2020-12-04T04:45:19.316500Z

I canā€™t reproduce this. Iā€™ll see if anyone else has the same problem and and see if I can see any pattern in whoā€™s getting it.

salam 2020-12-04T05:08:34.327400Z

thatā€™s strange. iā€™ll go back and try it again. this might just be me nitpicking but the other thing thatā€™s been bugging me about this run configuration window is the way some options are indented. i feel like the labels ā€œHostā€, ā€œPort,ā€ and ā€œContext Moduleā€ can use more indentation to make it clear that they are the children of the ā€œConnect to serverā€ radio button (same amount of indentation that the ā€œProjectā€ label has under the radio button ā€œUse port from nREPL fileā€.)

cfleming 2020-12-04T05:33:25.327600Z

Good idea, Iā€™ll fix that.

roklenarcic 2020-12-04T12:54:59.328400Z

Whatā€™s the trick for getting to connect to remote Babashka REPL? The connect process just waits

kenny 2020-12-04T15:26:55.328500Z

I see a new play button next to my deftests. If I click it I see several options: Run test, debug test, run with cpu profiler, ... If I click run with cpu profiler, I get a red popup in the corner pointing to the Run tab saying "Error running 'Test user/my-test': Internal error: Don't know how to patch selected configuration" Is running with cpu profiler not currently supported?

pez 2020-12-04T16:01:23.329300Z

Have you started babashka with an nrepl server?

borkdude 2020-12-04T16:01:51.329500Z

https://book.babashka.org/#repl and ctrl-f for Cursive

1šŸ˜
borkdude 2020-12-04T16:02:03.329800Z

if that info is out of date, please let me know

cfleming 2020-12-04T19:47:55.330Z

Iā€™m not sure about that actually, Iā€™ll check it. Iā€™m hoping to add coverage support, but itā€™s more complicated than I hoped because of how Clojure works.