Skip to content

Adding new article on Security best practices #3423

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

Merged
merged 5 commits into from
Apr 25, 2025

Conversation

Kenyatta-forbes
Copy link
Contributor

@Kenyatta-forbes Kenyatta-forbes commented Apr 25, 2025

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


@Copilot Copilot AI review requested due to automatic review settings April 25, 2025 19:49
@Kenyatta-forbes Kenyatta-forbes requested a review from a team as a code owner April 25, 2025 19:49
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new article outlining security best practices for projects, with detailed sections on multi-factor authentication, secure development workflows, dependency management, and vulnerability reporting.

  • Introduces key security themes such as MFA, secret scanning, and SCA tools.
  • Provides practical examples and guidelines for maintaining project security.
Comments suppressed due to low confidence (1)

_articles/security-best-practices-for-your-project.md:12

To gain insights into the experiences of maintainers and their strategies for finding balance, we ran a workshop with 40 members of the <a href="http://maintainers.github.com/">Maintainer Community</a>, allowing us to learn from their firsthand experiences...

@ahpook
Copy link
Contributor

ahpook commented Apr 25, 2025

Hey @Kenyatta-forbes this looks awesome! Great article.

There are some formatting lint failures that CI caught. It looks like most of them are due to "smart quotes" getting added by your editor, and there's a few where the formatter expects a blank line before a block quote or code sample: https://github.com/github/opensource.guide/actions/runs/14672360926/job/41181616740?pr=3423#step:6:272

Could you please take a pass at fixing these? I can help out if you want, LMK!

Adding spaces between lines

Signed-off-by: Kenyatta <[email protected]>
Copy link
Contributor Author

@Kenyatta-forbes Kenyatta-forbes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added spaces between lines. @ahpook I think I did it, lol.

updating `'`, not `’`  apostrophe 

Signed-off-by: Kenyatta <[email protected]>
Copy link
Contributor Author

@Kenyatta-forbes Kenyatta-forbes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

', not apostrophe - update

Copy link
Contributor Author

@Kenyatta-forbes Kenyatta-forbes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update ’

Copy link
Contributor Author

@Kenyatta-forbes Kenyatta-forbes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update to format

Copy link
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great article, thanks for the addition!

@Kenyatta-forbes Kenyatta-forbes added this pull request to the merge queue Apr 25, 2025
Merged via the queue into main with commit 4314ad6 Apr 25, 2025
7 checks passed
@Kenyatta-forbes Kenyatta-forbes deleted the New-Security-Article branch April 25, 2025 23:11
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