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

Added a list of weather conditions and cocktail names #278

Merged
merged 4 commits into from Sep 25, 2017

Conversation

Projects
None yet
3 participants
@thisisparker
Contributor

thisisparker commented Sep 21, 2017

Separately. Let me know if you would prefer me to submit these as two different PRs.

@hugovk

This comment has been minimized.

Show comment
Hide comment
@hugovk

hugovk Sep 21, 2017

Contributor

If you click through to the CI build, you can see it telex because it found some duplicates, like Heavy Drizzle Snow.

Please could you remove the repetitions?

Contributor

hugovk commented Sep 21, 2017

If you click through to the CI build, you can see it telex because it found some duplicates, like Heavy Drizzle Snow.

Please could you remove the repetitions?

@thisisparker

This comment has been minimized.

Show comment
Hide comment
@thisisparker

thisisparker Sep 21, 2017

Contributor

@hugovk yup, removed those! Strangely, those duplicates are in the source list from NOAA 🙄

Contributor

thisisparker commented Sep 21, 2017

@hugovk yup, removed those! Strangely, those duplicates are in the source list from NOAA 🙄

@dariusk dariusk merged commit d64a7f8 into dariusk:master Sep 25, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment