Bug#759311: RFS: node-base64-url/1.0.0-1 [ITP]

2014-08-26 Thread Riley Baird
Hi Joseph, Thanks for making the package for Debian! I'm not a DD, so I can't sponsor your package, but here's my review: d/control: -Don't put each dependency on a different line. -In your description, you don't need to include a description of what node.js is, rather, you can mention that your

Bug#759311: RFS: node-base64-url/1.0.0-1 [ITP]

2014-08-26 Thread Daniel Lintott
On 26 August 2014 08:46:27 BST, Riley Baird bm-2cvqnduybau5do2dfjtrn7zbaj246s4...@bitmessage.ch wrote: Hi Joseph, Thanks for making the package for Debian! I'm not a DD, so I can't sponsor your package, but here's my review: d/control: -Don't put each dependency on a different line. This is

Bug#759311: RFS: node-base64-url/1.0.0-1 [ITP]

2014-08-26 Thread Jérémy Lal
This package is questionnable because it contains too little code... https://github.com/joaquimserafim/base64-url/blob/master/index.js There is an ongoing discussion on pkg-javascript ML about bundling packages:

Bug#759311: RFS: node-base64-url/1.0.0-1 [ITP]

2014-08-26 Thread Riley Baird
d/control: -Don't put each dependency on a different line. This is actually common practice as having them on separate lines makes it easier to review a diff for future versions. Okay, thanks, I didn't know that. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org

Bug#759311: RFS: node-base64-url/1.0.0-1 [ITP]

2014-08-26 Thread Joseph Bisch
On Jérémy Lal kapo...@melix.org wrote: This package is questionnable because it contains too little code... https://github.com/joaquimserafim/base64-url/blob/master/index.js There is an ongoing discussion on pkg-javascript ML about bundling packages:

Bug#759311: RFS: node-base64-url/1.0.0-1 [ITP]

2014-08-25 Thread Joseph Bisch
Package: sponsorship-requests Severity: wishlist Dear mentors, I am looking for a sponsor for my package node-base64-url * Package name: node-base64-url Version : 1.0.0-1 Upstream Author : Joaquim Serafim * URL : https://github.com/joaquimserafim/base64-url * License