-
Notifications
You must be signed in to change notification settings - Fork 13
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
add contributing guide #85
Conversation
Signed-off-by: Ulises Gascón <[email protected]>
Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@UlisesGascon — Changes LGTM! Just one more phrasing fix and a suggested update to the current Code of Conduct.
Co-authored-by: lelia <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Signed-off-by: Ulises Gascón <[email protected]>
36f19d5
to
f3706d5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good draft, @UlisesGascon, and thanks for the initial review pass, @lelia!
Left some comments on content and structure.
Signed-off-by: Ulises Gascón <[email protected]>
a853f45
to
7cc7ce5
Compare
Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
…UlisesGascon/openssf-scorecard-monitor into docs/improve-contributing-guide
8d556b2
to
91f8bb9
Compare
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Co-authored-by: Stephen Augustus <[email protected]> Signed-off-by: Ulises Gascón <[email protected]>
Signed-off-by: Ulises Gascón <[email protected]>
c45d38e
to
1aa027b
Compare
Signed-off-by: Ulises Gascón <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after all the changes! 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff, @UlisesGascon!
Related #83