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

Upgrades for library compatibility #25

Closed
wants to merge 1 commit into from
Closed

Upgrades for library compatibility #25

wants to merge 1 commit into from

Conversation

petergoldstein
Copy link
Contributor

This pull request includes a few changes:

i) Eliminating use of the now removed should_change macro in the tests
ii) Updating the use of FactoryGirl to use the new syntax rather than the older syntax that is not supported in the latest versions of the FactoryGirl gem
iii) Editing the gemspec to support Rails versions greater than 3.0.x
iv) Adding a .travis.yml file

It would be great if this could be pulled in and a new gem version released. I'm looking to use this gem on a new Rails 3.2 project.

Please let me know if there's anything else I can do to make that happen. Thanks.

@petergoldstein
Copy link
Contributor Author

@tcocca Are you still maintaining this gem? Or should we look into forking this?

I'd really like to get a gem with several of these PRs pulled in. Thanks.

@seuros
Copy link

seuros commented Nov 1, 2013

+1

@petergoldstein
Copy link
Contributor Author

This PR has basically been outstripped by subsequent development. The master branch has been updated to Rails 4, FactoryGirl has been updated, etc. Closing.

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.

2 participants