Add deprecation warning when String#first and String#last receive neg…#33058
Merged
rafaelfranca merged 1 commit intorails:masterfrom Oct 2, 2018
Merged
Conversation
8bf8df0 to
ac5656a
Compare
ac5656a to
9d81130
Compare
ccc9402 to
824251d
Compare
…ative integers [Gannon McGibbon + Eric Turner]
824251d to
ec9a89c
Compare
suketa
added a commit
to suketa/rails_sandbox
that referenced
this pull request
Jun 15, 2019
Add deprecation warning when String#first and String#last receive negative integers rails/rails#33058
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…ative integers
Summary
This adds a deprecation warning when calling
String#firstandString#lastwith negative integer arguments. The current behaviour is the following:As you can see, it doesn't really line up with
Array's behaviour:See #29105 for the discussion.