这是indexloc提供的服务,不要输入任何密码
Skip to content

Fixed Code Quality Issues #398

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

Merged
merged 4 commits into from
Mar 1, 2021
Merged

Conversation

HarshCasper
Copy link
Contributor

Description

Summary:

  • Fixed Object Inheritance
  • Replace ternary syntax with if expression
  • Use literal syntax to create data structure
  • Add .deepsource.toml

I ran a DeepSource Analysis on my fork of this repository. You can see all the issues raised by DeepSource here.

DeepSource helps you to automatically find and fix issues in your code during code reviews. This tool looks for anti-patterns, bug risks, performance problems, and raises issues. There are plenty of other issues in relation to Bug Discovery and Anti-Patterns which you would be interested to take a look at.

If you do not want to use DeepSource to continuously analyze this repo, I'll remove the .deepsource.toml from this PR and you can merge the rest of the fixes. If you want to setup DeepSource for Continuous Analysis, I can help you set that up.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 1, 2021
@codecov
Copy link

codecov bot commented Mar 1, 2021

Codecov Report

Merging #398 (3c01500) into master (54bfc42) will decrease coverage by 0.12%.
The diff coverage is 93.10%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #398      +/-   ##
==========================================
- Coverage   89.32%   89.19%   -0.13%     
==========================================
  Files          13       13              
  Lines         721      731      +10     
==========================================
+ Hits          644      652       +8     
- Misses         77       79       +2     
Impacted Files Coverage Δ
googlemaps/places.py 94.21% <86.66%> (-1.37%) ⬇️
googlemaps/__init__.py 100.00% <100.00%> (ø)
googlemaps/client.py 92.12% <100.00%> (+0.11%) ⬆️
googlemaps/convert.py 96.66% <100.00%> (-0.03%) ⬇️
googlemaps/maps.py 95.45% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5253cbc...3c01500. Read the comment docs.

@jpoehnelt jpoehnelt merged commit 44cafa0 into googlemaps:master Mar 1, 2021
@jpoehnelt
Copy link
Contributor

Thanks!

googlemaps-bot pushed a commit that referenced this pull request Mar 1, 2021
## [4.4.3](v4.4.2...v4.4.3) (2021-03-01)

### Bug Fixes

* fixed code quality issues ([#398](#398)) ([44cafa0](44cafa0))
@googlemaps-bot
Copy link
Contributor

🎉 This PR is included in version 4.4.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement. released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants