Hi Xavier, On Wed, 07 Nov 2018, Xavier wrote: > I think teams should publish their own salsa.conf filesIf you agree, I > propose to stop adding feature to let Martin review the actual code and > add it to devscripts. Then after some tests in real life, we will be > able to continue adding new commands or options.
I tried to use the tool for pkg-security today and I stumbled on a few issues: - no way to enable the "Emails on push" integration which is the most important thing to configure for me - no support for the "Irker (IRC gateway)" integration either (we currently prefer the less verbose notifications of irker) See "Creating new repositories" on https://wiki.debian.org/Teams/pkg-security I also had SALSA_TOKEN="xxxxx" in a file referenced by SALSA_TOKEN_FILE in my main configuration file and this did not work, I had to stip the double quotes. The manual page does not document the value that we have to put into boolean variables (such as SALSA_TAGPENDING). I assume it's "yes/no" (or that yes/no should work). I have put this in my config file: SALSA_ENABLE_MR=yes Yet the tool always tells me that the variable merge_requests_enabled should be set to zero: $ salsa --conf-file +./pkg-security.conf check_repo --debug --verbose --all acct: merge_requests_enabled should be 0 [...] I used the last .deb that you built even if it's now older than the current version in sid. Cheers, -- Raphaël Hertzog ◈ Debian Developer Support Debian LTS: https://www.freexian.com/services/debian-lts.html Learn to master Debian: https://debian-handbook.info/get/
