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

Bug#960572: RFS: vim-ale/2.6.0-1 [ITP] (Asynchronous Lint Engine for Vim 8 and NeoVim), vim-vader/0.3.0+git20200213.6fff477-1 [ITP] (simple vimscript test framework)



Dear Nicholas,

Thanks for your work, I'll review it!

A few preliminary remarks:

on salsa's repo for vim-ale, you've created a debian/master branch that
is merely the same as upstream/latest for now, and a mymedia/master one
which seems to contain the debian packaging files.

I'd suggest you either remove mymedia/master in favour of debian/master
(it'd seem more relevant to upload the package from the content of
debian/master), or at least make the mymedia/master branch being the
default branch of the repository on salsa, and discard the debian/master
branch that seems to be useless.

The same goes for vim-vader.

My preference would be to rename mymedia/master to debian/master.

I'll handle both packages, but, next time, please open one RFS per
source package. The fact that both are closely related doesn't really
change a thing to that, and one bug for multiple packages makes it
harder to follow what has already been done and what is to be done.

Cheers.

-- 
Pierre-Elliott Bécue
GPG: 9AE0 4D98 6400 E3B6 7528  F493 0D44 2664 1949 74E2
It's far easier to fight for one's principles than to live up to them.

Attachment: signature.asc
Description: PGP signature


Reply to: