Skip to content

Conversation

@wustmz
Copy link

@wustmz wustmz commented Aug 15, 2022

No description provided.

@dataabc
Copy link
Owner

dataabc commented Aug 15, 2022

感谢热心贡献代码。第一个commit是可以的,关于第二个,我有一些建议:
1.新代码注释了一些初始化的代码,我不知道原因,如果确实不需要,可以去掉;
2.新代码增加了与邮件的交互,但是配置是在代码中,我认为在config文件中会更好;
3.代码中貌似有你的私人邮件和密钥,这是比较私密的东西,可能会给你的账号带来安全问题,为了邮件的安全,建议删除,可以使用example@email.com之类的替代。

这个只是我个人的理解,可能有不对的地方,再次感谢。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants