Closed pooja1pathak closed 9 months ago
CLA Assistant Lite bot All contributors have signed the CLA ✍️
@pooja1pathak I've approved your PR but it looks like CircleCI isn't running our tests. Did you run the whole test suite locally? Did it pass?
@c0c0n3 Yes I've checked and it passed. I have also checked on circle CI, only test_others is showing fail which is not related to my changes. Please find below detail of failing tests:
src/geocoding/tests/test_geocoding.py ......F.F...F. [ 93%]
@pooja1pathak thanks that's great. I think the problem w/ CircleCI is on our side---not quite sure why, but it might be related to the fact we're merging a branch from your fork since the build seems to work fine if the branch is in the main repo.
re: failing tests, those geocoding tests were failing even before your PR. So I'm merging your PR. Thank you so much for this contrib!!
Proposed changes
Removed 404 NOT FOUND in when no data found for query and returned 200 OK with empty set. Fix for issue #720 .
Types of changes
What types of changes does your code introduce to the project? Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.[x] Any dependent changes have been merged and published in downstream modules
"Martel Open Source Software Individual Contributor License Agreement" "Contributing to QuantumLeap" "QuantumLeap Release Notes"