Yes we try to note breaking changes in releases but this one obviously
slipped through. Sorry about that.

David

On Wed, May 31, 2017 at 7:03 PM, Daniel Compton <
daniel.compton.li...@gmail.com> wrote:

> I agree, it does make sense to consolidate to Clojure's behaviour. In the
> future it could be good to add an extra release notes section for changes
> like these that break the ClojureScript API contract.
>
> Thanks, Daniel.
>
> On Thu, Jun 1, 2017 at 10:53 AM David Nolen <dnolen.li...@gmail.com>
> wrote:
>
>> Thanks for the report. That's unfortunate but I think in this case
>> consolidating the behavior with Clojure is best.
>>
>> David
>>
>> On Wed, May 31, 2017 at 4:28 PM, Daniel Compton <
>> daniel.compton.li...@gmail.com> wrote:
>>
>>> Hi folks
>>>
>>> We ran into issues in production yesterday after upgrading to
>>> ClojureScript 1.9.456, and I haven't seen anyone else mention this so I
>>> thought I'd point it out for anyone upgrading.
>>>
>>> In versions prior to CLJS 1.9.456, seqable? tested only for the ISeqable
>>> protocol, i.e. collections. In CLJS-1875
>>> <https://dev.clojure.org/jira/browse/CLJS-1875>, released as part of
>>> 1.9.456
>>> <https://groups.google.com/forum/#!topic/clojurescript/wGJeG36GPTk> it
>>> now tests for anything that you can call seq on. This matches the
>>> behaviour of the seqable? function in Clojure that was recently added
>>> for 1.9, but was a breaking change in behaviour for ClojureScript.
>>>
>>> *In concrete terms, this means that strings and arrays that previously
>>> tested false for seqable? will now test true.*
>>>
>>> *In 1.9.229*
>>> cljs.user=> (seqable? [:a :b :c])
>>> true
>>> cljs.user=> (seqable? {:a :b :c :d})
>>> true
>>> cljs.user=> *(seqable? "abcd")*
>>> *false*
>>> cljs.user=> *(seqable? (into-array [:a :b :c]))*
>>> *false*
>>>
>>> *In 1.9.456 and beyond*
>>> cljs.user=> (seqable? [:a :b :c])
>>> true
>>> cljs.user=> (seqable? {:a :b :c :d})
>>> true
>>> cljs.user=> *(seqable? "abcd")*
>>> *true*
>>> cljs.user=> *(seqable? (into-array [:a :b :c]))*
>>> *true*
>>>
>>> --
>>> Daniel
>>>
>>> --
>>> Note that posts from new members are moderated - please be patient with
>>> your first post.
>>> ---
>>> You received this message because you are subscribed to the Google
>>> Groups "ClojureScript" group.
>>> To unsubscribe from this group and stop receiving emails from it, send
>>> an email to clojurescript+unsubscr...@googlegroups.com.
>>> To post to this group, send email to clojurescript@googlegroups.com.
>>> Visit this group at https://groups.google.com/group/clojurescript.
>>>
>> --
>> Note that posts from new members are moderated - please be patient with
>> your first post.
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "ClojureScript" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to clojurescript+unsubscr...@googlegroups.com.
>> To post to this group, send email to clojurescript@googlegroups.com.
>> Visit this group at https://groups.google.com/group/clojurescript.
>>
> --
> Note that posts from new members are moderated - please be patient with
> your first post.
> ---
> You received this message because you are subscribed to the Google Groups
> "ClojureScript" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to clojurescript+unsubscr...@googlegroups.com.
> To post to this group, send email to clojurescript@googlegroups.com.
> Visit this group at https://groups.google.com/group/clojurescript.
>

-- 
Note that posts from new members are moderated - please be patient with your 
first post.
--- 
You received this message because you are subscribed to the Google Groups 
"ClojureScript" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojurescript+unsubscr...@googlegroups.com.
To post to this group, send email to clojurescript@googlegroups.com.
Visit this group at https://groups.google.com/group/clojurescript.

Reply via email to