archlinuxcn / lilac

Lilac is the build bot for archlinuxcn
GNU General Public License v3.0
114 stars 40 forks source link

set emails in lilac.py #68

Closed petronny closed 5 years ago

petronny commented 6 years ago

目标:

PS. 如果lilac.py有错误那还是发给REPO.find_maintainer_or_admin...

lilydjwg commented 6 years ago

直接写 Email 地址是不是不太好?

我考虑的是写成:

maintainers = [{
  'github': 'lilydjwg',
}, {
  'archcn': 'xxx',
}, {
  'email': 'xxx',
}]

这样。可以发送到 github 上展示的邮箱、在维护者信息中登记的邮箱或者指定的邮箱。

Update: 也许可以写成 github:xxx 或者 email:xxx 这样的 URN?不过这样就需要做 URL encoding 了。

petronny commented 6 years ago

这。。。目的是为了防止email被爬取? 要不写成

emails = ['a at b.com']

然后手动加一个email.replace(' at ', '@')的转换。。。?

lilydjwg commented 6 years ago

你觉得写成那样就不会被爬取了吗? 另外,也可能有人不希望邮件地址被公开?

petronny commented 6 years ago

那就放个公钥出来加密吧😂 支持3种输入:

  1. email明文
  2. '@'转' at '的email
  3. 公钥加密后的email
lilydjwg commented 6 years ago

我觉得还是写 github id 最好了,因为这样 lilac 就不会找不到人来指派了。但是又有人没有在 github 里显示自己的邮箱,或者使用了不同的邮箱,所以又需要一种方式来指定邮箱。

felixonmars commented 6 years ago

可否考虑直接开 issue @人?这样一般都会收到邮件~

yuyichao commented 6 years ago

那样很多人都会收到提示

petronny commented 6 years ago

那就

maintainers = [{
  'github': 'lilydjwg',
}, {
  'email': 'xxx',
}, {
  'encrypted_email': 'xxx',
}]

lilydjwg commented 5 years ago

implemented in 6f709e52b80f01489c9cf08446bc84f303a1076b