| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Sorry, something went wrong.
|
@sondrelg Thanks for the PR! Please accept the CLA and I can merge |
Sorry, something went wrong.
|
It's not letting me sign the CLA for my private e-mail.. Can I just say that I accept it here? I accept! |
Sorry, something went wrong.
## [4.7.1](v4.7.0...v4.7.1) (2022-11-21) ### Bug Fixes * Convert print statement to info log ([#455](#455)) ([83ad827](83ad827))
|
🎉 This PR is included in version 4.7.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Sorry, something went wrong.
|
Hi all, this bit of the PR breaks the package: - print("\n","API queries_quota:", self.queries_quota,"\n")
+ logger.info("API queries_quota:", self.queries_quota)
A %s is needed in the message string, e.g. logger.info("API queries_quota: %s", self.queries_quota) Could anyone tackle this? Cheers |
Sorry, something went wrong.
|
I'm unavailable for the next few days unfortunately, but this seems like a 2 minute fix, so hopefully someone else can jump on it. Maybe you could open a PR @RemiCardona? |
Sorry, something went wrong.
|
@sondrelg sure thing, will do. Thanks for the quick response! |
Sorry, something went wrong.
|
@amuramoto Would you be able to take a look at this follow-up PR that corrects this log message from level INFO to DEBUG? |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Closes #451