Skip to content
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

GitHub: Add security policy #501

Merged
merged 6 commits into from
Jul 29, 2024
Merged

GitHub: Add security policy #501

merged 6 commits into from
Jul 29, 2024

Conversation

Lamparter
Copy link
Collaborator

No description provided.

@Lamparter Lamparter requested a review from 0x5bfa as a code owner July 28, 2024 19:16
@Lamparter Lamparter linked an issue Jul 28, 2024 that may be closed by this pull request
@Lamparter Lamparter self-assigned this Jul 28, 2024
@Lamparter Lamparter added this to the FluentHub v1.0.0 milestone Jul 28, 2024
@Lamparter Lamparter mentioned this pull request Jul 28, 2024
66 tasks
@0x5bfa

This comment was marked as resolved.

@Lamparter

This comment was marked as resolved.

@0x5bfa
Copy link
Owner

0x5bfa commented Jul 29, 2024

@Lamparter Now it looks good to me

@0x5bfa
Copy link
Owner

0x5bfa commented Jul 29, 2024

What do you think of commented out pieces, would we want to remove?
Otherwise, is it ready for merge?

@0x5bfa 0x5bfa added the triage/approved This PR has been approved by the team. label Jul 29, 2024
@Lamparter
Copy link
Collaborator Author

Yes I implemented everything I wanted to implement as far as I remember
For the commented lines let's just leave them in because they are the default guide comments for a security.md file and at some point we may want to use the Supported versions text

@Lamparter Lamparter merged commit 2919093 into main Jul 29, 2024
8 checks passed
@Lamparter Lamparter deleted the lamparter/security branch July 29, 2024 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/approved This PR has been approved by the team.
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

GitHub: Add SECURITY.md
2 participants