Skip to content

Add security policy link to readme #11331

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 9, 2021
Merged

Add security policy link to readme #11331

merged 1 commit into from
Mar 9, 2021

Conversation

per1234
Copy link
Collaborator

@per1234 per1234 commented Mar 9, 2021

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

@per1234 per1234 requested review from alranel, rsora and ubidefeo March 9, 2021 02:24
@per1234 per1234 added Component: Documentation Related to Arduino's documentation content feature request A request to make an enhancement (not a bug fix) security Security fixes / bugs / improvements labels Mar 9, 2021
ubidefeo
ubidefeo requested changes Mar 9, 2021
View reviewed changes
Copy link

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

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

Security is H3 while the other headers are H2

@per1234
Copy link
Collaborator Author

per1234 commented Mar 9, 2021

Hi @ubidefeo this was done intentionally. There is an existing "Bug Reports" section of the readme at H2, so I put the text about reporting security bugs as a subsection of that at H3.

You can see how it looks rendered here:
https://github.com/arduino/Arduino/blob/69cffd3c9cb3ffcca6adb767dc108e315a38d5c0/README.md#bug-reports-and-technical-discussions

@ubidefeo
Copy link

ubidefeo commented Mar 9, 2021

Acknowledged
Then this LGTM, I'll approve :)

ubidefeo
ubidefeo approved these changes Mar 9, 2021
View reviewed changes
Copy link

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

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

LGTM

@per1234 per1234 merged commit ed67cd3 into arduino:master Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ubidefeo ubidefeo ubidefeo approved these changes

@alranel alranel Awaiting requested review from alranel

@rsora rsora Awaiting requested review from rsora

Assignees
No one assigned
Labels
Component: Documentation Related to Arduino's documentation content feature request A request to make an enhancement (not a bug fix) security Security fixes / bugs / improvements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@per1234 @ubidefeo