Skip to content

Commit

Permalink
[IMP] Change pre-commit flake8 URL
Browse files Browse the repository at this point in the history
https://gitlab.com/pycqa/flake8 is currently blocked
for some countries and this gitlab repository is a mirror of
the github one https://github.com/pycqa/flake8
  • Loading branch information
Ernesto Tejeda committed Jun 1, 2022
1 parent 6e67b58 commit a48f386
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Expand Up @@ -25,7 +25,7 @@ repos:
- id: mixed-line-ending
args: ["--fix=lf"]
- id: trailing-whitespace
- repo: https://gitlab.com/pycqa/flake8
- repo: https://github.com/pycqa/flake8
rev: 3.9.2
hooks:
- id: flake8
Expand Down

4 comments on commit a48f386

@ap-wtioit
Copy link
Contributor

@ap-wtioit ap-wtioit commented on a48f386 Nov 15, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This now seems to affect Austria as well everyone: https://gitlab.com/pycqa/flake8 is giving me a 404.

How can we speed up propagating this commit to the repositories? (e.g. web still has the old URL)

Tested with VPN the URL seems no longer available anywhere (US, UK, Austria, Germany, Finland)

See also PyCQA/flake8#1737

@pedrobaeza
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was included in https://github.com/OCA/oca-addons-repo-template/releases/tag/v1.8.0, so it should be enough to apply copier -f update in the repo and do PR to update the definition.

@pedrobaeza
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm seeing that it's not being applied... It was only changed on the pre-commit of this repo itself, not on the template 🀦

@yajo
Copy link
Member

@yajo yajo commented on a48f386 Nov 15, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check out #170

Please sign in to comment.