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

Adding GH Actions Guides #42

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

tomgonzo
Copy link
Contributor

Created guides folder, added guides for oss, container, and iac

Modified top-level readme with a link to the guides above the existing examples.

Copy link
Collaborator

@garethr garethr left a comment

Choose a reason for hiding this comment

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

A few small nits. I left one comment each, but they apply to each of the guides in a few places.


You can use [Snyk](https://snyk.co/SnykGHGuide) to scan your applications' open source dependencies for security, license, and dependency health issues as part of your continuous integration \(CI\) workflow.

{% hint style="info" %}
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't appear to render correctly in the GitHub preview?

* `command`: defaults to `test`, specify which command to run
* `json` : defaults to `false`, save the results as `snyk.json`

```text
Copy link
Collaborator

Choose a reason for hiding this comment

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

Better to be ```yaml

@tomgonzo tomgonzo requested a review from a team as a code owner January 10, 2022 14:48
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