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

Use rails-5 branch for shoulda-matchers to remove deprecation warning #556

Merged
merged 1 commit into from
Jul 19, 2017
Merged

Use rails-5 branch for shoulda-matchers to remove deprecation warning #556

merged 1 commit into from
Jul 19, 2017

Conversation

medwards1771
Copy link
Contributor

See issue here: thoughtbot/shoulda-matchers#933
See fix here: thoughtbot/shoulda-matchers#943
DEPRECATION WARNING: #tables currently returns both tables and views. This behavior is deprecated and will be changed with Rails 5.1 to only return tables. Use #data_sources instead.

…ning

See issue here: thoughtbot/shoulda-matchers#933
See fix here: thoughtbot/shoulda-matchers#943
DEPRECATION WARNING: #tables currently returns both tables and views. This behavior is deprecated and will be changed with Rails 5.1 to only return tables. Use #data_sources instead.
@medwards1771 medwards1771 deleted the remove-deprecation-warnings branch July 11, 2017 15:34
@medwards1771 medwards1771 restored the remove-deprecation-warnings branch July 11, 2017 15:39
@medwards1771 medwards1771 reopened this Jul 11, 2017
@medwards1771
Copy link
Contributor Author

sorry, deleted that branch by accident.

@medwards1771
Copy link
Contributor Author

Also, i just realized my commit message may cause confusion since this project has a 'rails5' branch. The 'rails-5' branch I'm referring to in the message is a branch in the shoulda-matchers repo: https://github.com/thoughtbot/shoulda-matchers/tree/rails-5

@tjgrathwell tjgrathwell merged commit 98a892e into railsbridge:master Jul 19, 2017
@tjgrathwell
Copy link
Member

I normally try to avoid git-sourced gems, but who knows when a new proper release of shoulda-matchers is gonna come out. Let's do it. 😎

@medwards1771
Copy link
Contributor Author

Sweet ⚡️

@medwards1771 medwards1771 deleted the remove-deprecation-warnings branch July 19, 2017 17:55
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