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

FIX: Remove print command #26

Merged
merged 1 commit into from
Nov 29, 2023
Merged

FIX: Remove print command #26

merged 1 commit into from
Nov 29, 2023

Conversation

mgrover1
Copy link
Collaborator

Remove the print command for the version

Remove the print command for the version
@mgrover1 mgrover1 requested a review from zssherman November 29, 2023 18:49
@kmuehlbauer
Copy link
Collaborator

@mgrover1 Ah, I was almost about to check where this comes from 😬

@mgrover1 mgrover1 requested a review from kmuehlbauer November 29, 2023 19:04
@mgrover1
Copy link
Collaborator Author

👍 sorry about that @kmuehlbauer ! I added it in a previous PR and forgot to remove before committing 😄

@kmuehlbauer
Copy link
Collaborator

The nice thing is, this came up today on openradar discourse.

As wradlib is importing cmweather (if available) it looked like the wradlib import did the printout. The user was confused. 😂 Great, this will be fixed soon. 👍

@mgrover1 mgrover1 merged commit 8da50a9 into main Nov 29, 2023
7 checks passed
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