-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Add security advice to contrib guide #4187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add security advice to contrib guide #4187
Conversation
Remember… on GitHub you could actually enforce 2FA for the organization. |
Codecov Report
@@ Coverage Diff @@
## master #4187 +/- ##
=======================================
Coverage 19.97% 19.97%
=======================================
Files 153 153
Lines 30494 30494
=======================================
Hits 6091 6091
Misses 23489 23489
Partials 914 914 Continue to review full report at Codecov.
|
LGTM |
CONTRIBUTING.md
Outdated
@@ -201,6 +201,10 @@ an advisor has time to code review, we will gladly welcome them back | |||
to the maintainers team. If a maintainer is inactive for more than 3 | |||
months and forgets to leave the maintainers team, the owners may move | |||
him or her from the maintainers team to the advisors team. | |||
For security reasons, Maintainers should use 2FA for theirs accounts and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Little typo their's' accounts
Just a little reminder.
Related to #4167