search

EnterpriseyIntranet / nextcloud-API

NextCloud OCS API for Python
GNU General Public License v3.0
27 stars 27 forks source link

coding utf8, line length to 99 and indent improvements #37

Closed DylannCordel closed 5 years ago

DylannCordel commented 5 years ago

Hi,

We currently use your app with python 2.7. Seems ok (but utf8) for the usage we have. Adding explicit charset solve the python 2.7 problem.

codecov-io commented 5 years ago

Codecov Report

Merging #37 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   92.91%   92.91%           
=======================================
  Files          17       17           
  Lines         536      536           
=======================================
  Hits          498      498           
  Misses         38       38
Impacted Files Coverage Δ
src/nextcloud/response.py 84.84% <ø> (ø) :arrow_up:
src/nextcloud/__init__.py 100% <ø> (ø) :arrow_up:
src/nextcloud/api_wrappers/group_folders.py 100% <ø> (ø) :arrow_up:
src/nextcloud/api_wrappers/user.py 95.65% <ø> (ø) :arrow_up:
src/nextcloud/api_wrappers/apps.py 100% <ø> (ø) :arrow_up:
...rc/nextcloud/api_wrappers/federated_cloudshares.py 42.3% <ø> (ø) :arrow_up:
src/nextcloud/api_wrappers/capabilities.py 100% <ø> (ø) :arrow_up:
src/nextcloud/api_wrappers/notifications.py 91.66% <ø> (ø) :arrow_up:
src/nextcloud/api_wrappers/__init__.py 100% <ø> (ø) :arrow_up:
src/nextcloud/api_wrappers/user_ldap.py 100% <ø> (ø) :arrow_up:
... and 7 more

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 a1cbd46...eb75b0f. Read the comment docs.

scrutinizer-notifier commented 5 years ago

The inspection completed: No new issues

matejak commented 5 years ago

Merging, thank you for your contribution!