Skip to content

Rails 6.1: remove deprecation warnings #884

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

Conversation

mgrunberg
Copy link
Contributor

This PR deals with 4 deprecation warnings by looking at the deprecation commit and applying a similar update.

  1. allowed_index_name_length is deprecated and will be removed from Rails 6.2 (source rails/rails@ab2d859)
  2. [] is deprecated and will be removed from Rails 6.2 (Use configs_for) (source rails/rails@2a53fe6)
  3. Passing an Active Record object to directly is deprecated and will be no longer quoted as id value in Rails 6.2 (source rails/rails@87886c9)
  4. .reorder(nil) with .first / .first! no longer takes non-deterministic result in Rails 6.2. To continue taking non-deterministic result, use / instead. (source rails/rails@eec562d)

…eprecated and will be no longer quoted as id value in Rails 6.2

Deprecation commit
rails/rails@87886c9
…in Rails 6.2. To continue taking non-deterministic result, use / instead.

Deprecation commit
rails/rails@eec562d
@wpolicarpo wpolicarpo merged commit 4d0de64 into rails-sqlserver:main Apr 19, 2021
@mgrunberg mgrunberg deleted the issues/yellowspot/rails-6-1/remove-deprecation-warnings branch January 3, 2022 17:33
lavika pushed a commit to lavika/activerecord-sqlserver-adapter that referenced this pull request Sep 26, 2023
* DEPRECATION WARNING: allowed_index_name_length is deprecated and will be removed from Rails 6.2

Deprecation commit
rails/rails@ab2d859

* DEPRECATION WARNING: [] is deprecated and will be removed from Rails 6.2 (Use configs_for)

Deprecation commit
rails/rails@2a53fe6

* DEPRECATION WARNING: Passing an Active Record object to  directly is deprecated and will be no longer quoted as id value in Rails 6.2

Deprecation commit
rails/rails@87886c9

* DEPRECATION WARNING:  with  /  no longer takes non-deterministic result in Rails 6.2. To continue taking non-deterministic result, use  /  instead.

Deprecation commit
rails/rails@eec562d
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