-
Notifications
You must be signed in to change notification settings - Fork 7
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
fixes #31 change default logging level for precalculated to INFO #35
fixes #31 change default logging level for precalculated to INFO #35
Conversation
Hi @davemfish, I added you as a reviewer since you filed the issue! Let me know if you need any other changes or whatnot. |
Hi @emlys, do you mind taking a look at one more hopefully simple PR? I'd originally put this to Dave since he requested it but I just realized he's out for another week. Seems a long time to wait for a single line PR+history. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense to me! I'll just take a quick look at the merge conflict (and resolve if it's nontrivial), and then merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! Just waiting for the tests to pass post-merge conflict and then I'll merge.
🚀 |
@richpsharp I see you just re-requested a review from me post-merge ... did I miss something on the PR? I can revert the merge if needed! |
@phargogh, I didn't intentionally do that! Thanks for the merge. |
Changes the level of the "not precalculated" logging message from a WARNING to INFO.
Closes #31