summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* new release 0.10.21v0.10.21Arthur Zamarin2023-02-041-0/+71
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* PythonCheck: check for missing scm dep when neededArthur Zamarin2023-02-035-2/+61
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/537 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* Suppress the black reformat commit from git blameArthur Zamarin2023-02-031-0/+4
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* reblackify the codeArthur Zamarin2023-02-0325-59/+1
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* setup.py: fix usage of absolute pathArthur Zamarin2023-02-031-5/+5
| | | | | Bug: https://bugs.gentoo.org/892938 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* checks.python: small modernizationArthur Zamarin2023-01-311-16/+11
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* PythonPackageNameCheck: update docs & add testsArthur Zamarin2023-01-3110-5/+75
| | | | | Follows: 4438566500fdf116ba36c3c407022e89541867d6 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* checks: Add a check for Python pkgnames matching PyPIMichał Górny2023-01-311-0/+47
| | | | | | Signed-off-by: Michał Górny <mgorny@gentoo.org> Closes: https://github.com/pkgcore/pkgcheck/pull/534 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* InvalidMetadataRestrict: check for invalid restricts in metadata.xmlArthur Zamarin2023-01-264-0/+53
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* _GitCommitPkg: fix no attribute live or slotArthur Zamarin2023-01-241-0/+6
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/380 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* RdependChange: skip when revbumped in same batchArthur Zamarin2023-01-241-3/+3
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/459 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* SuspiciousSrcUriChange: fix false positive with mirrorsArthur Zamarin2023-01-212-12/+20
| | | | | | | | | | The original code for comparing for URL change was giving false positives when the URI is mirror based, since an object was used to compare. Pre-compute the URI string and use strings for comparison. Also add a test for that failure. Resolves: https://github.com/pkgcore/pkgcheck/issues/531 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* scan: fix quiet mode with keywordsArthur Zamarin2023-01-202-0/+9
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/413 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* GitPkgCommitsCheck: catch SRC_URI mistakesArthur Zamarin2023-01-202-2/+120
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/493 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* scan: suppress non-error results in quiet modeArthur Zamarin2023-01-202-1/+17
| | | | | | | When in quiet mode, include in scan only results of error level. Resolves: https://github.com/pkgcore/pkgcheck/issues/413 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* checks.git: add type annotationsArthur Zamarin2023-01-201-16/+13
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* EmptyDirsCheck: mark results' level as errorArthur Zamarin2023-01-201-3/+3
| | | | | | | | | This check is done at repo level, so if you are inside a package dir, it will be skipped. As a solution, mark those results as errors, so the bot will fail to regen metadata and we can notice the issue. Resolves: https://github.com/pkgcore/pkgcheck/issues/499 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* FormatReporter: mention `--format` in helpArthur Zamarin2023-01-201-2/+5
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/522 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* MissingManifest: fix behavior under thick reposArthur Zamarin2023-01-201-17/+17
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/530 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ArchesMisSync: check for arch.list <-> arches.desc missyncArthur Zamarin2023-01-204-0/+24
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/526 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* UnusedInherits: fix false positives with indirect usageArthur Zamarin2023-01-192-1/+25
| | | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/355 Resolves: https://github.com/pkgcore/pkgcheck/issues/527 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* Use REPO profile base for profile cachingDaniel M. Weeks2023-01-191-3/+5
| | | | | | | | | Prior to this change, scanning an overlay would always cause a profile cache update since nothing repo-specific was actually getting updated. Signed-off-by: Daniel M. Weeks <dan@danweeks.net> Closes: https://github.com/pkgcore/pkgcheck/pull/528 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ProfilesCheck: check missing values for implicit in make.defaultsArthur Zamarin2023-01-163-0/+37
| | | | | Related: https://github.com/pkgcore/pkgcheck/issues/524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ProfilesCheck: check USE_EXPAND_VALUES_* in make.defaultsArthur Zamarin2023-01-164-0/+48
| | | | | Related: https://github.com/pkgcore/pkgcheck/issues/524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ProfilesCheck: check for unknown USE_EXPAND_* in make.defaultsArthur Zamarin2023-01-165-8/+58
| | | | | Related: https://github.com/pkgcore/pkgcheck/issues/524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ProfilesCheck: check for unknown USE & IUSE_IMPLICIT in make.defaultsArthur Zamarin2023-01-154-1/+13
| | | | | Related: https://github.com/pkgcore/pkgcheck/issues/524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ProfilesCheck: check for unknown ARCH in make.defaultsArthur Zamarin2023-01-158-6/+33
| | | | | Related: https://github.com/pkgcore/pkgcheck/issues/524 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* VariableScopeCheck: Disallow D and ED in pkg_postinstUlrich Müller2023-01-141-2/+2
| | | | | | | | | | | | | According to QA policy PG 107: https://projects.gentoo.org/qa/policy-guide/ebuild-format.html#pg0107 (Note that "pkg_postinst" was misspelt as "pkg_postint", so it would have warned already.) Signed-off-by: Ulrich Müller <ulm@gentoo.org> Closes: https://github.com/pkgcore/pkgcheck/pull/523 Closes: https://github.com/pkgcore/pkgcheck/issues/516 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* EclassUsageCheck: check for setting user variables in ebuildsArthur Zamarin2023-01-123-1/+49
| | | | | | | Check ebuilds for overriding eclass' user variables. Resolves: https://github.com/pkgcore/pkgcheck/issues/512 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* MissingInherits: don't show for functions defined in ebuildArthur Zamarin2023-01-092-2/+23
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/513 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* add __main__.py fileArthur Zamarin2023-01-091-0/+4
| | | | | | | I'm adding this file, so you can call pkgcheck using `python -m pkgcheck`. Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* git checks: include revision for old name during rename opArthur Zamarin2023-01-092-1/+30
| | | | | | | | | | | | | | Previously it was including only the version without a revision, which was failing and crashing the git check. This fixes it by including the revision using the "fullver" property. Include a reproducer test which fails without this commit and passes with it. Reported-by: Sam James <sam@gentoo.org> Reported-by: David Seifert <soap@gentoo.org> Resolves: https://github.com/pkgcore/pkgcheck/issues/511 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ProvidedEclassInherit: new check for inheriting provided eclasesArthur Zamarin2023-01-024-32/+93
| | | | | Resolves: https://github.com/pkgcore/pkgcheck/issues/504 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* test_pkgcheck_replay: fix test_replay_pipe_stdin from sdistArthur Zamarin2022-12-312-30/+29
| | | | | | | | | We don't ship `bin/` in the sdist, so we are unable to run the script `bin/pkgcheck` from the sdist. Instead, use monkey-patching and `tool` to run with stdin `pkgcheck replay`. Bug: https://bugs.gentoo.org/888896 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* start work on 0.10.21Arthur Zamarin2022-12-311-1/+1
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* new release 0.10.20v0.10.20Arthur Zamarin2022-12-291-0/+8
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* ci: add black format checkArthur Zamarin2022-12-291-0/+7
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* format using blackArthur Zamarin2022-12-29130-5290/+6347
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* bump snakeoil minimal versionArthur Zamarin2022-12-271-3/+3
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* sync with pkgcore's error message formatArthur Zamarin2022-12-272-2/+2
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* update needed pkgcore versionArthur Zamarin2022-12-271-2/+2
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* Fix API compatibility w/ pkgcore revision eb6f4edd2Brian Harring2022-12-271-2/+1
| | | | | | | | | | | | | | | | | In that revision, USE flag validation is moved to eapi objects; EAPI ultimately arbitrates that, and it cleaned up some internal issues in atom. However, it broke pkgcheck's metadata check which was using that regex directly; that code's broken anyways- any demandloaded object must not be imported into another scope; the lazy replacement only affects the original scope, thus that `self.valid_use = ` was pinning the proxied regex. Either way; since EAPI objects now have a use flag validation method, use that. Resolves: https://github.com/pkgcore/pkgcheck/issues/502 Signed-off-by: Brian Harring <ferringb@gmail.com> Closes: https://github.com/pkgcore/pkgcheck/pull/503 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* fix docs generation styleArthur Zamarin2022-12-153-8/+14
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* MissingRemoteIdCheck: give ready <remote-id/> sampleMichał Górny2022-12-131-2/+2
| | | | | | | | | Instead of verbose description of what to put in the missing remote-id element, just print a ready-to-use XML snippet. Signed-off-by: Michał Górny <mgorny@gentoo.org> Closes: https://github.com/pkgcore/pkgcheck/pull/500 Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* start work on 0.10.20Arthur Zamarin2022-12-021-1/+1
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* new release 0.10.19v0.10.19Arthur Zamarin2022-11-261-0/+21
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* readd updated tox.iniArthur Zamarin2022-11-261-0/+21
| | | | Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
* PythonCheck: stop warning about eclass use on python:2.7 depsMichał Górny2022-11-267-66/+22
| | | | | | | The python2_7 target support are phased out from the eclass, so stop warning about direct dependencies. Signed-off-by: Michał Górny <mgorny@gentoo.org>
* PythonCheck: replace obsolete 2_7 target with 3_5 in testsMichał Górny2022-11-241-91/+91
| | | | Signed-off-by: Michał Górny <mgorny@gentoo.org>
* PythonCheck: stop testing for pypy(2.7)Michał Górny2022-11-242-13/+0
| | | | | | | The eclasses no longer support the pypy target, so stop checking for dev-python/pypy. Signed-off-by: Michał Górny <mgorny@gentoo.org>