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

Update gems and upgrade rails to 4.1.8 #66

Merged
merged 11 commits into from
Jan 7, 2015
Merged

Update gems and upgrade rails to 4.1.8 #66

merged 11 commits into from
Jan 7, 2015

Conversation

kookster
Copy link
Member

updates all gems and rails.
includes minor change from ignore -> transient for factory girl factories (ignore is deprecated)

@farski
Copy link
Member

farski commented Jan 7, 2015

CI is passing but coveralls seems to be having some issue; anything to worry about?

[Coveralls] Submitting to https://coveralls.io/api/v1
Coveralls encountered an exception:
RestClient::UnprocessableEntity
422 Unprocessable Entity

@cqr
Copy link
Contributor

cqr commented Jan 7, 2015

@farski fixing now, thanks.

@cqr
Copy link
Contributor

cqr commented Jan 7, 2015

@farski should be fixed, coveralls was lagging on the repo name change.

farski added a commit that referenced this pull request Jan 7, 2015
@farski
Merge pull request #66 from PRX/fix/update_gems_rails_418
2666222 
Update gems and upgrade rails to 4.1.8
@farski farski merged commit 2666222 into master Jan 7, 2015
@cqr cqr deleted the fix/update_gems_rails_418 branch August 5, 2015 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@farski farski

Labels
reviewplx
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kookster @farski @cqr