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

Remove references to Rails 4 from assets guide [ci skip] #23560

Conversation

prathamesh-sonpatki
Copy link
Member

No description provided.

from other gems. For example, jquery-rails includes a copy of jquery.js
and enables AJAX features in Rails.

The asset pipeline is technically no longer a core feature of Rails 4, it has
The asset pipeline is technically no longer a core feature from Rails 4 onwards, it has
Copy link
Contributor

Choose a reason for hiding this comment

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

while the PR is open, can you change the comma to a "--"?

@prathamesh-sonpatki
Copy link
Member Author

@maclover7 done :)

@@ -21,11 +21,11 @@ What is the Asset Pipeline?
The asset pipeline provides a framework to concatenate and minify or compress
JavaScript and CSS assets. It also adds the ability to write these assets in
other languages and pre-processors such as CoffeeScript, Sass and ERB.
It allows assets in your application to be automatically combined with assets
It allows assets in your application to be automatically combined with assets
Copy link
Member

Choose a reason for hiding this comment

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

no changes here.

Copy link
Member Author

Choose a reason for hiding this comment

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

It's whitespace change and do we allow such changes when we are updating a file without having too much git churn.

@prathamesh-sonpatki
Copy link
Member Author

Merging this, thanks for the review everyone.

prathamesh-sonpatki added a commit that referenced this pull request Feb 8, 2016
…sets-guide

Remove references to Rails 4 from assets guide [ci skip]
@prathamesh-sonpatki prathamesh-sonpatki merged commit 9dcf67c into rails:master Feb 8, 2016
@prathamesh-sonpatki prathamesh-sonpatki deleted the rm-rails-4-from-assets-guide branch February 8, 2016 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants