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

Update README.md #127

Merged
merged 2 commits into from
Jan 10, 2023
Merged

Update README.md #127

merged 2 commits into from
Jan 10, 2023

Conversation

ChronicLynx
Copy link
Contributor

@ChronicLynx ChronicLynx commented Jan 9, 2023

Correct toml config example to avoid error "Key 'IgnoredVulns.id' has already been defined."

The existing example will produce an error if you try to use it as is to build your config file. Ran into this issue today when implementing it into a project.

@google-cla
Copy link

google-cla bot commented Jan 9, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Correct toml config example to avoid error "Key 'IgnoredVulns.id' has already been defined."
@another-rex
Copy link
Collaborator

/gcbrun

@another-rex
Copy link
Collaborator

Thanks for the fix!

@another-rex another-rex merged commit a7f1d04 into google:main Jan 10, 2023
julieqiu pushed a commit to julieqiu/osv-scanner that referenced this pull request May 2, 2023
Correct toml config example to avoid error "Key 'IgnoredVulns.id' has already been defined."
julieqiu pushed a commit to julieqiu/osv-scanner that referenced this pull request May 2, 2023
Correct toml config example to avoid error "Key 'IgnoredVulns.id' has already been defined."
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