A few improvements :) #2

Closed
wants to merge 8 commits into
from

3 participants

@sylvinus

No description provided.

@dachev

It doesn't seem like this module is actively maintained. This puts people who want to use it in a very difficult place. Please relinquish npm rights to someone who is willing to fix bugs.

@brettstimmerman

Hi dachev, funny that I see your note only a few minutes after pushing some changes.

I'll be addressing sylvinus's pull requests shortly. Some of them have been implemented in 0.0.2 which should hit npm today.

@brettstimmerman

Dependecy path error fixed in 2a54e02; better reporting for config file errors fixed in 976ba92.

I plan on implementing absolute filenames once I'm comfortable everything else is working ok. Version 0.0.2 is more about making sure everything in 0.0.1 still works since a lot has changed in node and the dependencies in the last few months.

I want to think about how best to include Sass support. It may be a good candidate for a plugin, rather than out-of-the-box support. In general, though, it's a great idea.

I also want to think about the chainable compressors/handlers idea a bit more, but again I think it's a good idea.

@dachev

Excellent news. Thanks a lot.

@brettstimmerman

I'm cooking up new ideas at the moment related to pluggable support for processors and CDNs, as well as soldering a mixed-type component in a single pass. For example, soldering Sass and CSS files, minifying the combined result, and pushing to CDN.

Initial testing is looking good and I hope to push some of these changes to a new branch soon. For the most part, the API will remain the same, but there will be some changes to the config format to support the new features.

@brettstimmerman

Hey @sylvinus, I've let this PR rot far too long, and I'm not maintaining solder anymore.

If it's still a thing you'd like to do, let me know. Otherwise, I'll be closing this in a few days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment