<div id="geary-body" dir="auto"><div><br></div></div><div id="geary-quote" dir="auto"><br>On Mon, Oct 7, 2019 at 18:32, Nicolas Mora <nicolas@babelouest.org> wrote:<br><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">Indeed it's just warning messages, I've read the build log too fast, the
error was because of invalid tests.
The upstream package uses karma to run tests so I had to disable testing
in the package build. I let a note as a reminder for the future in
debian/tests/pkg-js/test

I've made other changes in the package to use pkg-js-tools features.
</div></blockquote><span style="white-space: pre-wrap;"><br></span><div><span style="white-space: pre-wrap;">* You can change pkg-js-tools dependency to (>= 0.8.13~) and then remove debian/install file.</span></div><div><span style="white-space: pre-wrap;"><br></span></div><div><span style="white-space: pre-wrap;">* You remove unnecessary comments like #override.. #export... in rules.</span></div><div><span style="white-space: pre-wrap;"><br></span></div><div><span style="white-space: pre-wrap;">* It is better to remove debian/tests/pkg-js/test instead of creating a dummy script. You can ignore that lintian warning (it makes it easy to know if a package is not running tests). Since no tests are run node-tape could be removed from build dependencies.</span></div><div><span style="white-space: pre-wrap;"><br></span></div><div><span style="white-space: pre-wrap;">* You can push to https://salsa.debian.org/js-team/node-i18next/ after squashing related commits.</span></div><div><br></d
 iv><div><br></div></div>