I don't quite know what I mean here, but I have a general sense of unease with the current setup, so it deserves at least a critical eye before making it into a real release.
Off the top of my head:
Having to derive from the standard library is pretty awkward, but there should also be a way to avoid having all the Sass functions built in.
Having to explicitly declare the number of arguments is kind of silly when most of the time we just want to use the Python argspec. It's also tedious to declare every possible number of arguments when there are lots of defaults, and I'm not even sure how named arguments work here (if at all?). Consider waiting on #131 and automagically examining the Python argspec by default?
Maybe passing in a library should be replaced with passing in extensions a la #130? If you only want to add functions, just create an extension that only contains functions.
eevee
commented
10 years ago
I don't quite know what I mean here, but I have a general sense of unease with the current setup, so it deserves at least a critical eye before making it into a real release.
Off the top of my head: