superlou / fountain-atom

Grammar and tools for the Fountain screenwriting syntax
MIT License
48 stars 15 forks source link

Replaced flatten with extend #81

Closed superlou closed 6 years ago

superlou commented 6 years ago

This appears to address the issue in #69 (and a few others). I'm not sure how this was working before, so I'm nervous about merging this quickly without confirmation that this was really the root of the problem.

jrbostic commented 6 years ago

I'll do some validation asap.

jrbostic commented 6 years ago

Underscore-plus was pulling in underscore 1.6.0 until June of this year. And then they did a patch release that updated the underscore version to 1.8.3. Flatten was rewritten somewhere in between. Case closed.

I like your code better anyway. :) Works great!

jrbostic commented 6 years ago

Also, there's always the option to nail down dependency versions to avoid breaks like that... we'd just be giving up automatically staying current.

Another thing is that the spec tests actually failed on this, so if you ran them locally before the release and they passed, its probably because you had a stale node_modules entry for underscore-plus. Tricky.

jrbostic commented 6 years ago

And that's certainly why I couldn't reproduce the problem...

superlou commented 6 years ago

Awesome sleuthing. I'm going to release a patch tonight.

I noticed the spec tests failing when I pulled down a fresh copy of the package. It's probably worth looking at pinning down versions of packages we rely on, since even if we can catch a regression via tests, most users can't. I also might look into our use of underscore. If it's only one or two places for basic functionality, it might be able to fairly cleanly just address it with CoffeeScript syntactic sugar. Either way, I'm going to release the fix as soon as I can.

Do you want to be added as a collaborator for this repository?

jrbostic commented 6 years ago

Oh what's in a name!? But yea, I'd love to be a collaborator! :D

superlou commented 6 years ago

I actually thought you already were one. Invite sent.

jrbostic commented 6 years ago

Accepted! Mwahaha. ;)