Skip to content

Adding new article on Security best practices#3423

Merged
Kenyatta-forbes merged 5 commits intomainfrom
New-Security-Article
Apr 25, 2025
Merged

Adding new article on Security best practices#3423
Kenyatta-forbes merged 5 commits intomainfrom
New-Security-Article

Conversation

@Kenyatta-forbes
Copy link
Copy Markdown
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 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
Copy Markdown
Contributor

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
Copy Markdown
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 <153775386+Kenyatta-forbes@users.noreply.github.com>
Copy link
Copy Markdown
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 <153775386+Kenyatta-forbes@users.noreply.github.com>
Copy link
Copy Markdown
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

update - ’

Signed-off-by: Kenyatta <153775386+Kenyatta-forbes@users.noreply.github.com>
Copy link
Copy Markdown
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 ’

update to format

Signed-off-by: Kenyatta <153775386+Kenyatta-forbes@users.noreply.github.com>
Copy link
Copy Markdown
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
Copy Markdown
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.

3 participants