-
Notifications
You must be signed in to change notification settings - Fork 53
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
middleman-sprockets fails to start throwing an error #130
Comments
It looks like at some point during the middleman 4.0 development, the method middleman-sprockets is using as a v4 check changed names via changelog:
If you're game to switch that out & test it'd be super helpful. Also, since you are running middleman 4, I'd suggest updating to using middleman-sprockets 4+ since it has been developed much more recently with middleman 4 compat in mind. |
I'm trying to upgrade to version 4. However I get errors in require
statements because for some reason sprockets is unable to find the
specified files..
Is something changed to that regard?
Thanks
…On Feb 14, 2018 18:45, "Steven Sloan" ***@***.***> wrote:
It looks like at some point during the middleman 4.0 development, the
method middleman-sprockets is using as a v4 check changed names via
changelog
<https://github.com/middleman/middleman/blob/09f3be9aba1ee07d5afcf8601e368f1e8fa8c524/CHANGELOG.md#400beta1>
:
rename app.add_to_config_context to Extension.expose_to_config for adding
extension-local methods to the sandboxed scope of config.rb
https://github.com/middleman/middleman-sprockets/blob/v3-
stable-real/lib/middleman-sprockets/extension.rb#L48
If you're game to switch that out & test it'd be super helpful.
Also, since you are running middleman 4, I'd suggest updating to using
middleman-sprockets 4+ since it has been developed much more recently with
middleman 4 compat in mind.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#130 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAUkPWuF-YIbXw_UQIKcOTe2DCVbONPMks5tUxuogaJpZM4SDttZ>
.
|
IIRC adding paths to sprockets changed a little bit, the upgrade guide may help. If you can share your |
@stevenosloan Thanks for the response :) Here are the information requested: These are the versions of
This is the relevant part of
Where
In fact, I've installed PreloadJS at
However in a source CoffeeScript file I've defined a require statement of
That gives error:
I would be really glad to you if you can help me.. Thanks |
One thing I see is that there doesn't appear to be a
that'll be what you want. |
Expected behavior and actual behavior
middleman-sprockets
fails to start throwing the following error:Steps to reproduce the problem (from a clean middleman installation) or example repo
Gemfile:
Additional information
2.5.0
4.1.10
v3-stable-real
The text was updated successfully, but these errors were encountered: