Fork me on GitHub
#babashka-sci-dev
<
2022-04-03
>
Bob B16:04:38

I might just be missing some history, but I'm concerned I'm missing a file or something - sci's native test script has a performance test that runs tests from sci.performance-test - in my local and in the CI run I looked at from yesterday, that invocation runs 0 tests, 0 assertions. Based on everything I can see, there are currently no performance tests to be run, but I'm wondering: ā€¢ was this test ns removed intentionally? (just want to make sure it wasn't an accident to be addressed) ā€¢ should we remove that test to eliminate a lein invocation? (or is it there so if performance tests come back, they'll get run without having to remember to re-add it)

borkdude16:04:00

They might have been removed, don't worry about that

borkdude16:04:21

And yes, if that file isn't there anymore, the lein invocation can go

teodorlu20:04:38

Hi šŸ‘‹

šŸ‘‹ 1
teodorlu20:04:31

PR: Add neil dep add entrypoint. Exactly the same as neil dep add - but fits with neil dep versions. https://github.com/babashka/neil/pull/19

borkdude21:04:02

Merged :)

šŸŽ‰ 1
teodorlu21:04:47

Would you like CI too before cutting a release?

borkdude21:04:14

@teodorlu I published version 0.0.18 now. A CHANGELOG.md and CI would be swell. Also feel free in #announcements what you've done.

šŸ‘€ 1
borkdude21:04:57

About CI: I usually use https://github.com/turtlequeue/setup-babashka - works pretty well, but just bash installer script works too. Merged.

teodorlu21:04:56

Ah, nice. Didn't know there was an existing action for adding babashka.

borkdude21:04:41

Should the help reflect neil dep add as well?

teodorlu21:04:53

Good catch.

borkdude21:04:55

I'm tuning out for today, thanks a lot!

teodorlu21:04:37

My pleasure!

borkdude21:04:10

Awesome announcement. Perhaps that fzf info can also end up in the README

1
šŸ™Œ 1
borkdude21:04:44

This is really beautiful:

$ neil dep versions org.clojure/clojure | fzf | xargs neil dep add

1
teodorlu21:04:09

Unix pipes are great šŸ˜„

borkdude22:04:53

If you want to make a tweet about this, with your video, I'll RT ;)

1
teodorlu10:04:36

@U04V15CAJ I guess I should have pinged you about that one ā˜ļø šŸ˜„

borkdude10:04:16

oh thanks :)

teodorlu22:04:16

Looks like I forgot to run bb run gen-script for https://github.com/babashka/neil/pull/21.

borkdude22:04:48

@teodorlu No problem, that'll happen automatically when I run bb publish again.

šŸ‘ 1