Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Cleanup and remove unused dependencies #154

Merged
merged 3 commits into from
Apr 6, 2021
Merged

Cleanup and remove unused dependencies #154

merged 3 commits into from
Apr 6, 2021

Conversation

jesec
Copy link
Contributor

@jesec jesec commented Apr 3, 2021

No description provided.

robertsLando
robertsLando approved these changes Apr 6, 2021
View reviewed changes
Copy link
Contributor

@robertsLando robertsLando left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Absolutely LGTM. Also I like removing the places.json file as it makes the code easier to read

@jesec jesec merged commit b47c260 into master Apr 6, 2021
@jesec jesec deleted the pr/cleanup branch April 6, 2021 08:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@robertsLando robertsLando robertsLando approved these changes

@hipstersmoothie hipstersmoothie Awaiting requested review from hipstersmoothie

@leerob leerob Awaiting requested review from leerob

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jesec @robertsLando