Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

Downgrade sprockets #28

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

Conversation

mfbx9da4
Copy link

@mfbx9da4 mfbx9da4 commented Jul 1, 2015

CSS injection was not working for newest rails as sprockets was naming files as self.js and browsersync was not picking files with this name.

```
cd gulp-rails-pipeline
bundle install
```

If you are on mac you may be required to run the following to [install PostgreSQL libraries successfully](http://stackoverflow.com/a/28206009/1376627).
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't include PostgreSQL as a dependency here... this bit might be distracting/confusing to anyone not using it.

Copy link
Author

Choose a reason for hiding this comment

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

To be honest I'm not sure why I had to install it in this way or why I needed the pg gem. Could be a mac thing. I'll look into it further and update it accordingly.

Maybe the readme should say something like run bundle install --without production for those who don't have postgresql installed?

@greypants
Copy link
Contributor

Thanks for opening this! Just a couple questions about the README updates.

@@ -8,3 +8,4 @@ gem 'jquery-rails'
gem 'turbolinks'
gem 'sqlite3', group: :development
gem 'pg', group: :production # For heroku
gem 'sprockets', '2.11.0'
Copy link
Author

Choose a reason for hiding this comment

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

Maybe this could just be a requirement for test and development?

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

Successfully merging this pull request may close these issues.

None yet

2 participants