docs: adds documentation for scorecard certifier - #216
Conversation
✅ Deploy Preview for resonant-wisp-1a517a ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
7bec021 to
13f7a23
Compare
funnelfiasco
left a comment
funnelfiasco
left a comment
There was a problem hiding this comment.
This looks really good overall. There are a few edits worth making before we merge.
13f7a23 to
1dec0a6
Compare
funnelfiasco
left a comment
funnelfiasco
left a comment
There was a problem hiding this comment.
Looks good to me! Once guacsec/guac#2791 is merged, I'll merge this.
|
Observed review from funnelfiasco+445852 (@funnelfiasco) |
|
Observed review from mihaimaruseac+323199 (@mihaimaruseac) |
|
@funnelfiasco this is ready to merge, now, right? |
Not to my knowledge. The PR that implemented some of the features (guacsec/guac#2791) was closed without merging. I'm not sure if it was done in a subsequent PR instead. @gaganhr94? |
|
The PR guacsec/guac#2815 is currently open, and needs reviews. Once this is merged, we can consider merging this docs PR as well |
✅ Deploy Preview for resonant-wisp-1a517a ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Hi maintainers, the PR guacsec/guac#2815 has been merged in GUAC. This documentation is also updated with the improvements made to the scorecard certifier. The PR is ready for review now. |
|
@gaganhr94 you'll need to fix the formatting with I'm not entirely sure what the netlify errors are about. |
b9252ef to
1629fca
Compare
Signed-off-by: Gagan H R <hrgagan4@gmail.com>
Signed-off-by: Gagan H R <ghr@guidewire.com>
b8533ab to
501cdee
Compare
|
@funnelfiasco fixed all the issues related to npx prettier and netlify errors |
3 participants
Fixes #215
These changes add documentation for scorecard certifier