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.

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

Adding first pass at the SECURITY-INSIGHTS.yml #4872

Merged
merged 3 commits into from
Oct 21, 2023
Merged

Adding first pass at the SECURITY-INSIGHTS.yml #4872

merged 3 commits into from
Oct 21, 2023

Conversation

jkowall
Copy link
Contributor

@jkowall jkowall commented Oct 20, 2023

Which problem is this PR solving?

  • Adding first pass at SECURITY-INSIGHTS.yml file

Description of the changes

  • Adding file based on the project configuration to satisfy new requirements from CLOMonitor

How was this change tested?

  • No testing needed as this file is not used internally to the project

Checklist

@jkowall
Adding first pass at the SECURITY-INSIGHTS.yml
7019568 
Signed-off-by: Jonah Kowall <jkowall@kowall.net>
@jkowall jkowall requested a review from a team as a code owner October 20, 2023 20:37
@jkowall jkowall requested a review from joe-elliott October 20, 2023 20:37
yurishkuro
yurishkuro reviewed Oct 20, 2023
View reviewed changes
SECURITY-INSIGHTS.yml Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Oct 20, 2023 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

see 2 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

yurishkuro
yurishkuro reviewed Oct 20, 2023
View reviewed changes
SECURITY-INSIGHTS.yml Outdated Show resolved Hide resolved
@jkowall
Fixing a few issues per feedback on SECURITY-INSIGHTS.yml
8e93c7d 
Signed-off-by: Jonah Kowall <jkowall@kowall.net>
@jkowall
Changing badge on README.md for OpenSSF
a457b73 
Signed-off-by: Jonah Kowall <jkowall@kowall.net>
@yurishkuro yurishkuro added the changelog:ci Change related to continuous integration / testing label Oct 21, 2023
@yurishkuro yurishkuro merged commit 7aeb457 into jaegertracing:main Oct 21, 2023
32 of 34 checks passed
@achrinza achrinza mentioned this pull request Oct 27, 2023
Open
@jkowall jkowall deleted the security-insights branch October 31, 2023 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yurishkuro yurishkuro yurishkuro approved these changes

@joe-elliott joe-elliott Awaiting requested review from joe-elliott joe-elliott is a code owner automatically assigned from jaegertracing/jaeger-maintainers

Assignees
No one assigned
Labels
changelog:ci Change related to continuous integration / testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jkowall @yurishkuro