Skip to content

Ruby 2.3's #dig#3458

Closed
kares wants to merge 6 commits intomasterfrom
test-2.3-dig
Closed

Ruby 2.3's #dig#3458
kares wants to merge 6 commits intomasterfrom
test-2.3-dig

Conversation

@kares
Copy link
Member

@kares kares commented Nov 12, 2015

[].dig(0) and {}.dig(:a) as introduced by MRI 2.3.0.preview1

@eregon
Copy link
Member

eregon commented Nov 13, 2015

@kares Nice work and specs here!
Could you make a PR against ruby/rubyspec?
That would avoid a possible spec duplication and to test the specs on multiple MRI versions 😃
I can then merge quickly the specs back to jruby master.

@kares
Copy link
Member Author

kares commented Nov 13, 2015

@eregon thx, I thought the work-flow is that you pick those eventually back to ruby/rubyspec.
so than I should remove those [rubyspec] commits from here ? ... also not sure how long this would be open before getting merged - maybe its only to be commited after 2.3 final as JRuby 9.1 will come out

@eregon
Copy link
Member

eregon commented Nov 13, 2015

@kares For new feature specs I think it's more beneficial to PR against the ruby/rubyspec repo.
Specs for existing stuff is less critical and can more easily be merged later.
Also, this allows to tweak the spec while it's getting stable in MRI.

A guard of minimal version 2.3 will be needed anyway for this spec to pass on older versions, so this spec will only run when JRuby declares RUBY_VERSION to be "2.3.x" (it's possible to ignore such a guard with --unguarded while developing).

Yes, please remove them from here if you do the PR so there is no chance for conflict :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing a terminating newline character here.

@kares kares mentioned this pull request Nov 15, 2015
@kares kares added this to the JRuby 9.1.0.0 milestone Nov 21, 2015
@kares kares added the ruby 2.3 label Nov 21, 2015
@headius headius mentioned this pull request Nov 21, 2015
58 tasks
@kares
Copy link
Member Author

kares commented Nov 21, 2015

put the commits into ruby-2.3 which should merge to master after 2.3 final when 9.1.0 is released

@kares kares closed this Nov 21, 2015
@headius headius deleted the test-2.3-dig branch December 15, 2016 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants