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

Add timestamps to package urls #263

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

bpaul
Copy link

@bpaul bpaul commented Jan 8, 2015

No description provided.

nathanstitt and others added 14 commits September 23, 2013 13:25
For now work around by by making our own tag.  Pull request
rails/rails#11707 will correct on next version
of Rails
It doesn't appear to work with MiniTest
Jammit uses Rails tags_with_options for css tags,
which no longer not includes the `type="text/css"` attribute
This way it matches Rails
Also convert a few instances of just assert foo==bar to use
assert_equal so a better error message is generated
When Sass implemented source map support in version 3.4,
the signature of Sass::SCSS::CssParser.new changed to require
3 arguments
Since Rails has not been "booted", it doesn't setup the "url_for" method
which is called internally by the default url_options method.

Over-ridding it with a blank definition allows the controller's methods
to be tested
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.

3 participants