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

PR: Support coma and tab separated-value weather station list #32

Merged
merged 22 commits into from
Sep 9, 2017

Conversation

jnsebgosselin
Copy link
Owner

Fixes #31

Also added a test for this feature

@pep8speaks
Copy link

pep8speaks commented Sep 8, 2017

Hello @jnsebgosselin! Thanks for updating the PR.

Line 28:1: E722 do not use bare except'
Line 791:17: E722 do not use bare except'
Line 803:17: E722 do not use bare except'

Line 14:1: E402 module level import not at top of file

Comment last updated on September 09, 2017 at 05:06 Hours UTC

@coveralls
Copy link

Coverage Status

Coverage increased (+2.07%) to 2.849% when pulling 8bd1f39 on weather_sta_list_data_sep_coma into 67be197 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+6.9%) to 7.634% when pulling 409e50c on weather_sta_list_data_sep_coma into 67be197 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+6.9%) to 7.634% when pulling 009c655 on weather_sta_list_data_sep_coma into 67be197 on master.

@jnsebgosselin jnsebgosselin merged commit 2cded73 into master Sep 9, 2017
@jnsebgosselin jnsebgosselin deleted the weather_sta_list_data_sep_coma branch September 9, 2017 13:53
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.

3 participants