Skip to content

Fixes & updates for compass-sinatra sample project#1

Open
adamesque wants to merge 6 commits into
chriseppstein:masterfrom
adamesque:master
Open

Fixes & updates for compass-sinatra sample project#1
adamesque wants to merge 6 commits into
chriseppstein:masterfrom
adamesque:master

Conversation

@adamesque
Copy link
Copy Markdown

Hey Chris,

Here's a handful of fixes to hopefully make this Sinatra sample project a little easier to get started with.

Removing the compiled css file from /static is probably the most helpful change, because if it's present, Sinatra will serve that instead of recompiling the source sass file.

The other potentially helpful change swaps out sass syntax for scss. Not only do most of the tutorials these days seem to use scss, but I, uh, couldn't even get the sass version working. That might just be me not being familiar enough with sass to update it properly.

The other changes are mostly cosmetic.

Left unresolved by this pull request:

  • Running compass compile will once again prevent Sinatra from ever recompiling, by regenerating the static file
  • It seems like the "right" way to integrate Sass with Sinatra these days is to use Sass's Rack plugin, but that might be a little advanced for this sample project.

Thanks,
Adam

Adam Luikart and others added 6 commits April 30, 2011 17:30
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.

2 participants