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

Remove unused Hashie dependency #31

Merged
merged 1 commit into from
Mar 31, 2021

Conversation

andyw8
Copy link
Owner

@andyw8 andyw8 commented Mar 31, 2021

This was added in 59e3d43 but there's no reference to Hashie anywhere in the code.

cc @turadg in case I've missed some reason this is needed.

@andyw8 andyw8 force-pushed the andyw8/remove-unused-hashie-dependency branch from 602da85 to e0f9938 Compare March 31, 2021 13:11
@andyw8 andyw8 force-pushed the andyw8/remove-unused-hashie-dependency branch from e0f9938 to a661988 Compare March 31, 2021 13:13
Copy link
Collaborator

@turadg turadg left a comment

Choose a reason for hiding this comment

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

LGTM. Looks to have been a vestige from some local iteration before pushing.

@andyw8 andyw8 merged commit af0e3d2 into master Mar 31, 2021
@andyw8 andyw8 mentioned this pull request Apr 2, 2021
@andyw8 andyw8 deleted the andyw8/remove-unused-hashie-dependency branch April 11, 2021 19:11
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