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

Convert Mash keys for #dig #349

Merged
merged 1 commit into from Feb 8, 2016
Merged

Convert Mash keys for #dig #349

merged 1 commit into from Feb 8, 2016

Conversation

k0kubun
Copy link
Contributor

@k0kubun k0kubun commented Feb 6, 2016

In Ruby 2.3.0, I want to use Hash#dig with symbol arguments for Mash.

@marshall-lee
Copy link
Contributor

👍 but please wrap everything related to dig under if RUBY_VERSION >= '2.3.0', not only spec example. I don't want def dig in < 2.3.0

@k0kubun
Copy link
Contributor Author

k0kubun commented Feb 7, 2016

Fixed so and squashed in 93b3fe5.

@marshall-lee
Copy link
Contributor

@k0kubun okay, good, now please increase Metrics/ClassLength in .rubocop_todo.yml to the value large enough to pass a check. Or (i like this variant more) disable Metrics/ClassLength completely. I believe Hashie is not that sort of projects where this metric matters.

@k0kubun
Copy link
Contributor Author

k0kubun commented Feb 7, 2016

Thank you for your comment since I'm not familiar with rubocop. Increased Metrics/ClassLength for now.

@dblock
Copy link
Member

dblock commented Feb 7, 2016

For rubocop I generally just run rubocop -a ; rubocop --auto-gen-config.

This needs an entry in CHANGELOG, please.

@k0kubun
Copy link
Contributor Author

k0kubun commented Feb 8, 2016

Added CHANGELOG entry and squashed the commits.

@dblock
Copy link
Member

dblock commented Feb 8, 2016

Merging. Consider adding this in other extensions where you think it's useful!

dblock added a commit that referenced this pull request Feb 8, 2016
Convert Mash keys for #dig
@dblock dblock merged commit 6cfa234 into hashie:master Feb 8, 2016
@k0kubun k0kubun deleted the dig branch February 8, 2016 23:32
jsonn pushed a commit to jsonn/pkgsrc that referenced this pull request Oct 15, 2016
[3.4.6]: hashie/hashie@v3.4.5...v3.4.6

### Fixed

* [#368](hashie/hashie#368): Since `hashie/mash` can be required alone, require its dependencies - [@jrafanie](https://github.com/jrafanie).

## [3.4.5] - 2016-09-16

[3.4.5]: hashie/hashie@v3.4.4...v3.4.5

### Added

* [#337](hashie/hashie#337), [#331](hashie/hashie#331): `Hashie::Mash#load` accepts a `Pathname` object - [@gipcompany](https://github.com/gipcompany).

### Deprecated

* [#366](hashie/hashie#366): Hashie is no longer tested on Ruby < 2 - [@dblock](https://github.com/dblock).

### Fixed

* [#358](hashie/hashie#358): Fixed support for Array#dig - [@modosc](https://github.com/modosc).
* [#365](hashie/hashie#365): Ensured ActiveSupport::HashWithIndifferentAccess is defined before use in #deep_locate  - [@mikejarema](https://github.com/mikejarema).

### Miscellanous

* [#366](hashie/hashie#366): Added Danger, PR linter - [@dblock](https://github.com/dblock).

## [3.4.4] - 2016-04-29

[3.4.4]: hashie/hashie@v3.4.3...v3.4.4

### Added

* [#349](hashie/hashie#349): Convert `Hashie::Mash#dig` arguments for Ruby 2.3.0 - [@k0kubun](https://github.com/k0kubun).

### Fixed

* [#240](hashie/hashie#240): Fixed nesting twice with Clash keys - [@bartoszkopinski](https://github.com/bartoszkopinski).
* [#317](hashie/hashie#317): Ensure `Hashie::Extensions::MethodQuery` methods return boolean values - [@michaelherold](https://github.com/michaelherold).
* [#319](hashie/hashie#319): Fix a regression from 3.4.1 where `Hashie::Extensions::DeepFind` is no longer indifference-aware - [@michaelherold](https://github.com/michaelherold).
* [#322](hashie/hashie#322): Fixed `reverse_merge` issue with `Mash` subclasses - [@marshall-lee](https://github.com/marshall-lee).
* [#346](hashie/hashie#346): Fixed `merge` breaking indifferent access - [@docwhat](https://github.com/docwhat), [@michaelherold](https://github.com/michaelherold).
* [#350](hashie/hashie#350): Fixed from string translations used with `IgnoreUndeclared` - [@marshall-lee](https://github.com/marshall-lee).
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.

None yet

3 participants