PR linters
Open
@silkeh is already working on this.
Since Sep 20, 2023.
Priority: Wishlist
- Dominant language
- Python
- Stars
- 141
- Forks
- 146
- Avg merge
- 11h 7m
- Merged PRs (30d)
- 407
Description
Add GitHub integration(s) to detect common problems with PRs:
- Included
.eopkgfiles (#377). - Included
Makefile(#377). - Files in the wrong directory (ie
packages/*andpackages/*/*) (#377). - Use of
patch <instead of-i(#377). - Missing
homepageinpackage.yml(#377). - Package is not bumped once (as a warning) (#475).
- Initial release on inclusion != 1 (#475).
- Incorrect package component (depends on #370).
- Alphabetized build dependencies (
pkgconfig,A-Z,a-z,0-9) (#695). - The commit must not end with a
](#889). - Version numbers that are valid numbers must have quotations (#963).
- pspec_x86_64.xml is invalid (cannot be parsed).
- Licences are valid SPDX qualifiers (#888).
- Check if there is a metadata.xml file for font packages. Lint the file.
- Appstream metadata (see below)
-
abi_used_libsfor packages insystem.baseandsystem.develmust be part ofsystem.baseorsystem.devel(https://github.com/getsolus/packages/pull/891). - Hardcoded python version (
python3\.\d+) inpackage.yml.
To be refined further:
- Single package bump per commit.
- PR build order.
- Package layout.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.