search

fedora-infra / anitya

A cross-distribution upstream release monitoring project
https://release-monitoring.org
GNU General Public License v2.0
245 stars 104 forks source link

pylint anitya/lib/backends/cgit.py #1612

Closed lmilbaum closed 1 year ago

softwarefactory-project-zuul[bot] commented 1 year ago

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci https://fedora.softwarefactory-project.io/zuul/buildset/b9bd425f138c4cb4bde678c5dd86d0c3

:heavy_check_mark: fi-tox-mypy SUCCESS in 10m 35s :heavy_check_mark: fi-tox-lint SUCCESS in 10m 11s :heavy_check_mark: fi-tox-format SUCCESS in 10m 03s :heavy_check_mark: fi-tox-python38 SUCCESS in 14m 20s :heavy_check_mark: fi-tox-python39 SUCCESS in 14m 16s :heavy_check_mark: fi-tox-python310 SUCCESS in 14m 00s :heavy_check_mark: fi-tox-python311 SUCCESS in 14m 05s :heavy_check_mark: fi-tox-docs SUCCESS in 11m 16s :heavy_check_mark: fi-tox-bandit SUCCESS in 9m 59s :x: fi-tox-diff-cover FAILURE in 15m 18s

lmilbaum commented 1 year ago

@Zlopez Same coverage error as before. Thanks.

Zlopez commented 1 year ago

@lmilbaum No problem. I should probably add pylint to tox as well, so at least you can see the errors before submitting the PR.

Zlopez commented 1 year ago

I will probably do it, after you fix all the pylint errors and the pre-commit check will go through.

lmilbaum commented 1 year ago

I will probably do it, after you fix all the pylint errors and the pre-commit check will go through.

I'll do it if you don't mind :-)

lmilbaum commented 1 year ago

Please approve this PR

Zlopez commented 1 year ago

I will probably do it, after you fix all the pylint errors and the pre-commit check will go through.

I'll do it if you don't mind :-)

If you want to do it, I'm no against it :-)