Thank you for your upload! I did a quick review of the package in the
NEW queue and I do have a few things I'd like cleared out before we get
this one accepted into universe:

1) debian/watch file seems to be incorrect? I would like this accepted
before proceeding. At least when I tried running uscan I always got this
warning and no upstream tarball has been downloaded:

uscan warn: In debian/watch no matching hrefs for version 1.1.9 in watch line
  https://github.com/sadko4u/lsp-plugins/tags .*/release-?(\d\S*)\.tar\.gz

2) I still see that override_dh_auto_test disables tests. In comment #6
you mentioned that you were somehow able to make those working. Do those
really not work currently? Of course if there's no way of running them
during build then that's fine, but I want to make sure that they're
disabled for valid reasons. Especially that there was some discussion
regarding that previously.

3) This here is not a problem per se, but just something I was curious
about. In debian/control, in the long descriptions, I see that there's a
link to documentation. Is this official documentation for the plugins?
The webpage seems to be different than the home-page. Also, it seems to
be all in French - which is fine, but I guess it would be best if for
non-localized versions of the descriptions the docs was in English. It's
fine if there's nothing like that.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1827288

Title:
  [Needs Packaging] LSP-Plugins for Eoan

To manage notifications about this bug go to:
https://bugs.launchpad.net/lsp-plugins/+bug/1827288/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to