This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2022-09-08
Channels
- # announcements (40)
- # babashka (14)
- # babashka-sci-dev (7)
- # beginners (50)
- # calva (8)
- # cider (25)
- # clj-kondo (7)
- # cljdoc (8)
- # cljs-dev (14)
- # clojars (6)
- # clojure (56)
- # clojure-australia (1)
- # clojure-berlin (2)
- # clojure-dev (16)
- # clojure-europe (18)
- # clojure-nl (1)
- # clojure-norway (5)
- # clojure-uk (7)
- # clojurescript (100)
- # cursive (57)
- # data-science (9)
- # datomic (6)
- # emacs (11)
- # figwheel (2)
- # fulcro (14)
- # helix (2)
- # hyperfiddle (9)
- # introduce-yourself (1)
- # lsp (20)
- # malli (14)
- # meander (34)
- # minecraft (1)
- # missionary (8)
- # off-topic (37)
- # pedestal (4)
- # polylith (18)
- # portal (3)
- # re-frame (5)
- # ring (33)
- # shadow-cljs (32)
- # spacemacs (6)
- # vim (16)
Hey @alexmiller, I'm curious about your approach (and the clojure core team in general) to deciding which bugs to fix or even what counts as a bug. For example, I ran into the issue detailed in https://clojure.atlassian.net/browse/CLJ-2640 - "ex-info should be tolerant of nil data param" where an argument with map destructuring is nil
and is passed to ex-info
. I handled it as described in the ticket (`(ex-info "message" (or m {}))`) but it felt like a place where ex-info
could just "do the right thing". What led you to the decision to not merge the patch? Or request an alternative version of the patch that makes the change in a new way.
just hasn't gone through the process yet
My apologies, I don't mean to ask directly about that one necessarily, I'm purely curious about your process. Hoping to better understand it as I recently added a patch for clojure.test and don't have a good frame of reference for how things work
If it's helpful, I'll pop this in the chat in case it's helpful: https://clojure.org/dev/workflow
that's the overall process but of course there are vastly more things to look at than we will actually have time to look at, so I assume your question is how do we pick
Fogus and I build target lists to go through with Rich - those are a combination of things highly voted on ask clojure, things of high priority (based on other time swept sweeping through big issue lists and prioritizing), and generally things people regularly ask about
we've built some lists for 1.12, Rich hasn't looked at any of it yet
On the subject of 1.12, I still have regular reminders popping up about that stability issue I reported with the alpha but haven't yet had the space to put a version of the problematic app in production with the alpha in order to get thread and heap dumps. I will get to it. Maybe next week.
I did find one bad bug in the alpha1 code https://clojure.atlassian.net/browse/CLJ-2718
can turn a finite repeat into an infinite repeat, no idea if you happened to hit that
I wouldn't have expected that to be the cause but I guess it is possible. Strange that it affects only one app, as far as we could tell, although we do have a (drop 10 countries)
in that app where countries
is a lazy seq that could have less than 10 elements in some situations.
We also have a (drop n segments)
call where n
varies from 1
to (inc (count segments))
-- but none of those involve repeat
(or can this bug surface with other drop calls?
it's specific to repeat
OK, then it wouldn't affect that app.
yeah, seemed unlikely
I didn't realize every single patch still had to get explicit approval from Rich, that's interesting. Cool, thanks for the explanation