[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#816611: RFS: yamllint/1.0.3-1 [ITP] -- A linter for YAML files



Paul, thanks for this complete answer.

> Since I am interested in more static analysis and linting tools for
> check-all-the-things I'm willing to sponsor this, but only in return
> for a patch for check-all-the-things adding support for any one of the
> tools with TODO comments or checks flagged todo.

OK. Please find 4 enhancement commits (they are signed):

git fetch https://github.com/adrienverge/check-all-the-things.git enhancements
git log 7017206..FETCH_HEAD

(The "how to contribute section" suppose contributors have write
access to collab-maint, but this is not my case).

> In addition, you may want to submit pull requests to the linters and
> git-lint repos to advertise yamllint more widely and get it supported
> by git-lint.

Done for `linters`, will do for `git-lint`.

> The package fails to build in a clean chroot, see below for the log.

Thanks, fixed upstream and repackaged as 1.0.4-1.
https://mentors.debian.net/package/yamllint

> Please sign your tags, commits and releases using OpenPGP and have
> uscan verify them tarball signatures.

I might sign upstream commits in the future, but not right now.

> debian/copyright has unknown@unknown as your email address.

Fixed.

> Did you really mean for the license for debian/ to be different to the
> rest of the code? debian/copyright says GPL-2+ for it but GPL-3+ for
> the rest.

Fixed.

> The Standards-Version in debian/control is out of date, please read
> the upgrading guide and bump it:

Upgraded.

> Please uncomment the Vcs-* fields and point them at the VCS for the
> Debian packaging.

Done.

> You may want to wrap-and-sort the debian meta-data for easier to read
> diffs.

Done.

> Please add some upstream metadata: https://wiki.debian.org/UpstreamMetadata

Done.


Reply to: