search

akretion / roulier

API for package delivery
http://akretion.github.io/roulier/
GNU Affero General Public License v3.0
22 stars 21 forks source link

remove useless old py27 # -*- coding: utf-8 -*- #147

Closed DylannCordel closed 3 years ago

DylannCordel commented 3 years ago

some cleanup of old python 2.7 comment to set the charset used for files. In py3, charset is utf8 by default.

florian-dacosta commented 3 years ago

@DylannCordel We use pre-commit with some black config to format the code. If you can run it to fix the linting issues, it would be great.

DylannCordel commented 3 years ago

@florian-dacosta done, but there is a strange travis error I don't understand.

DylannCordel commented 3 years ago

@florian-dacosta the error seems to come from master. It occurs on my 3 PR I just rebased. Maybe due to travis environnement because I dont have this error on my local machine.

florian-dacosta commented 3 years ago

How fast! Thanks Yes, I saw this error, it occured when I merged a PR earlier today, but seems unrelatd to the merged PR. I'll try to fix it

florian-dacosta commented 3 years ago

@DylannCordel Seems that this is the jinja2 recent version that were an issue. If froze it for now to avoid the issue.