Skip to content

Remove orderedhash dep#18

Open
nikushi wants to merge 5 commits into
darwin:masterfrom
nikushi:remove-orderedhash-dep
Open

Remove orderedhash dep#18
nikushi wants to merge 5 commits into
darwin:masterfrom
nikushi:remove-orderedhash-dep

Conversation

@nikushi
Copy link
Copy Markdown

@nikushi nikushi commented Jan 1, 2016

Hi,

In my understanding, we can remove the dependency for orderedhash since ruby(>= 1.9) supports ordered hash.

Note: this branch was forked from #17

@jeremywrnr
Copy link
Copy Markdown

I just installed this gem, and needed to install orderedhash in order to run it, since the latest published gem still requires it (lib/csv2json.rb:4). So I suggest either removing its usage and requirement, or adding it as a formal requirement to the gemspec.

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