Skip to content

Conversation

@0xTim
Copy link
Member

@0xTim 0xTim commented Jun 16, 2021

To comply with new SSWG guidelines

Copy link
Collaborator

@fabianfett fabianfett left a comment

Choose a reason for hiding this comment

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

LGTM. However there is also a Security.md in this repo, which is good because it is linked to by GitHub (Security tab). Are we sure in this case that we want to link out to the vapor main repo?

@0xTim
Copy link
Member Author

0xTim commented Jun 16, 2021

@fabianfett where's the security.md in this repo? I couldn't see it. It should be the one in vapor/.github which gets linked to all repos

@fabianfett
Copy link
Collaborator

Ah! TIL. Okay. All green!

@0xTim 0xTim merged commit 8527fef into main Jun 16, 2021
@0xTim 0xTim deleted the add-security branch June 16, 2021 10:33
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