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

Sprockets 3 support #246

Merged
merged 9 commits into from
Oct 23, 2015
Merged

Sprockets 3 support #246

merged 9 commits into from
Oct 23, 2015

Conversation

robkilby
Copy link
Contributor

This PR builds upon the work done by @urkle in his fork. I’ve also provided fixes for the lack of Hike in later builds of Sprockets.

All tests are passing locally on both builds of Sprockets 2 and Sprockets 3.

Rob Kilby added 7 commits August 25, 2015 13:22
Based on #236, I’m bumping the version to 3.0.0
#trail no longer exists, and the functionality in Sprockets 3 now
exists in the Sprockets::Environment class (and its cached sibling).
#trail is a protected method.
This should allow Sprockets 2 to be installed for Rails 3
@thenanyu
Copy link

We would very much love this to get merged. I believe being stuck on an old version of sprockets exposes us to this: sstephenson/sprockets#441

@eitoball
Copy link

+1 for merging this PR. I am using this branch without any problems.

This commit changes the root of the file cache to match the default in sprockets-rails and write to the app-level tmp folder.
This also has the added benefit of segregating asset caches by environment.
@robkilby
Copy link
Contributor Author

This should fix #232, #236, and #244 - @craigmcnamara any comments?

@mib32
Copy link

mib32 commented Aug 28, 2015

@robkilby That's really cool but i get

undefined method `=~' for #<Pathname:../images/icons/categories/animals.png>

when trying to make a sprite!

@robkilby
Copy link
Contributor Author

Thanks for the feedback @mib32. I'll look into the issue over the weekend.

@mib32
Copy link

mib32 commented Aug 28, 2015

@robkilby thanks, i'm also working on this

@mib32
Copy link

mib32 commented Aug 28, 2015

@robkilby Forget that, it is a bug in compass-core, which is already deprecated, so it's impossible to fix. I'll use sprite_factory for my sprites!

Reflects the cache directory fix.
@matsuhisa
Copy link

I also am using this branch without any problems.

@quiqueg
Copy link

quiqueg commented Sep 15, 2015

Any update on getting this merged?

@NealJMD
Copy link

NealJMD commented Sep 23, 2015

I've been using gem "compass-rails", git: 'https://github.com/robkilby/compass-rails', branch: 'sprockets-3' in my Gemfile. It works great, and it would be great to get it merged and released so I can clean that up.

Thanks!

@Orforio
Copy link

Orforio commented Sep 23, 2015

Same here, +1 for this PR!

craigmcnamara added a commit that referenced this pull request Oct 23, 2015
@craigmcnamara craigmcnamara merged commit a1c8906 into Compass:master Oct 23, 2015
@robkilby robkilby deleted the sprockets-3 branch October 23, 2015 18:46
@CyborgMaster
Copy link

Now that this is merged, can we get a release to RubyGems?

@quiqueg
Copy link

quiqueg commented Jan 8, 2016

+1 for a new release

@rmm5t
Copy link
Contributor

rmm5t commented Jan 10, 2016

👍 New release, please.

@Davidiusdadi
Copy link

Could it be that Sprockets 3 support is still buggy? See #256

@aeberlin
Copy link

Could we please get a new 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.