Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Task to compile Less #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pfig
Copy link

@pfig pfig commented Mar 11, 2012

Hi,

I've added a task to compile Less (which I assume will be at the same level as the css directory for the theme).

I'm not 100% happy with having to edit the Rakefile to set the theme, but hey, this is for hackers, right?

Thank you so much for your work in JB, I now have a company website that doesn't suck :)

Cheers,

Pedro


SOURCE = "."
LESS = File.join(SOURCE, "assets", "themes", "twitter") # set theme here
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestions welcome for dealing with this. Read in _config.yml, perhaps? It seems a lot of work just to get a variable. It's a pain because the theme is jekyll-land, whereas the Rakefile is JB-land.

@plusjade
Copy link
Owner

@pfig this looks great for LESS users but I can only require a gem dependency if it significantly a part of a core feature.

What you can do is take everything you've written here and add it to its own file then load it in using this line: https://github.com/plusjade/jekyll-bootstrap/blob/master/Rakefile#L308

Basically any files in _rake with the extension of .rake will get loaded into the Rakefile environment. Your feature is the perfect use case.

If and when you get this in please repost here. Then I'll add a note about it in the documentation website so users can download your code.

What do you think?

@pfig
Copy link
Author

pfig commented Mar 25, 2012

Oh wow, I hadn't noticed that before, talk about looking at a tree and missing the forest :)

Yes, that sounds good, thank you! I'll create a gist and post the link here later today.

@pfig
Copy link
Author

pfig commented Mar 25, 2012

@plusjade
Copy link
Owner

Thanks for the quick work, @pfig . I will get this properly documented on http://jekyllbootstrap.com as soon as I can. Once it's documented and referenced, I'll close the issue by providing the link.

I appreciate you taking the time to contribute - thanks!

@mushon
Copy link

mushon commented Oct 7, 2014

@pfig, any chance you can you instruct me on using this? I added the '''_rake/lessc.rake''' to the site root. what next?

(thanks)

@groundh0g groundh0g added this to the v 0.5.0 milestone Apr 4, 2015
@groundh0g groundh0g self-assigned this Apr 4, 2015
@groundh0g groundh0g modified the milestones: v 0.5.0, v 0.6.0 Oct 18, 2015
@groundh0g
Copy link
Collaborator

Moving Rakefile tasks to new v 0.6.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants