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.

Sign up for GitHub

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

Jump to bottom

process_arguments spec: options argument seems to be no longer a Dict.t #78

Closed
weaktyper opened this issue Feb 28, 2016 · 1 comment
Closed

process_arguments spec: options argument seems to be no longer a Dict.t #78

weaktyper opened this issue Feb 28, 2016 · 1 comment

Comments

@weaktyper
Copy link

On line 92
The options Dict.t in the spec is not longer supported since the Keyword module is being used to parse the options now rather than the Dict module. I was using maps for the options those are no longer supported. Seems to be the [Keyword] type in the latest version I think.

recent changes
@weaktyper weaktyper changed the title process_arguments spec seems to be no longer a Dict.t process_arguments spec: options argument seems to be no longer a Dict.t Feb 28, 2016
@valpackett
Copy link
Owner

Thanks!

Used Dialyzer on the project. It didn't complain about that (but I've changed the type to [{atom(), any()}]) – however it caught a bug in #77 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@valpackett @weaktyper