search

codeforamerica / amiinlv

Am I In Las Vegas? Sometimes it's hard to know, because the city limits have the complexity of a fractal curve.
http://amiinlasvegas.com/
BSD 3-Clause "New" or "Revised" License
10 stars 5 forks source link

Proposed fix for: Insert LESS compilation step into npm make script #20 #34

Closed preppypiet closed 10 years ago

preppypiet commented 10 years ago

Hi, this is my proposed fix for "Insert LESS compilation step into npm make script #20". It uses gulp to do the less and browserify builds. I've updated the readme with instructions on how to use it.

louh commented 10 years ago

Nice. Thanks for the suggestion!

I'll merge this in, and do some cleanup on the master branch (e.g. moving LESS files to /src, linking to the generated CSS file rather than the LESS file and loading the LESS parser JavaScript, etc.)

:clap:

preppypiet commented 10 years ago

It's no problem at all, it seemed fine with the new build system but please just double check and make sure everything still works as expected. I ran it through its paces and there were no script or styling errors that I could see, so it should be fine.

On Thu, Jul 31, 2014 at 2:23 AM, Lou Huang notifications@github.com wrote:

Nice. Thanks for the suggestion!

I'll merge this in, and do some cleanup on the master branch (e.g. moving LESS files to /src, linking to the generated CSS file rather than the LESS file and loading the LESS parser JavaScript, etc.)

[image: :clap:]

— Reply to this email directly or view it on GitHub https://github.com/codeforamerica/amiinlv/pull/34#issuecomment-50702475.