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.

Sign up for GitHub

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

Jump to bottom

Use most recent sprockets manifest #1064

Merged
merged 1 commit into from
Apr 22, 2018
Merged

Use most recent sprockets manifest #1064

merged 1 commit into from
Apr 22, 2018

Conversation

justin808
Copy link
Member

@justin808 justin808 commented Apr 21, 2018 •
edited
Loading

This change is Reviewable

@justin808 justin808 mentioned this pull request Apr 21, 2018
Closed
@justin808 justin808 force-pushed the use-most-recent-sprockets-manifest branch from 355a63c to a61508d Compare April 21, 2018 23:28
@justin808
Sprockets: Use the most recent manifest
05955bc 
When creating symlinks, use the most recent manifest found.

See #1023
@justin808 justin808 force-pushed the use-most-recent-sprockets-manifest branch from a61508d to 05955bc Compare April 22, 2018 00:15
@coveralls
Copy link

coveralls commented Apr 22, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at ?% when pulling 05955bc on use-most-recent-sprockets-manifest into 0cb3768 on master.

@justin808 justin808 merged commit 098b2d2 into master Apr 22, 2018
@justin808
Copy link
Member Author

@mapreal19 please review.

@justin808 justin808 requested a review from mapreal19 April 22, 2018 00:50
@justin808 justin808 deleted the use-most-recent-sprockets-manifest branch April 22, 2018 01:45
@mapreal19
Copy link
Contributor

:lgtm:

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mapreal19 mapreal19 Awaiting requested review from mapreal19

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@justin808 @coveralls @mapreal19