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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃尡 Add script to set up probe boilerplate #3948

Merged
merged 7 commits into from May 8, 2024

Conversation

raghavkaul
Copy link
Contributor

What kind of change does this PR introduce?

Adds script to set create a new directory for a probe and set up *.go files with boilerplate for a probe.

Signed-off-by: Raghav Kaul <raghavkaul+github@google.com>
Copy link

codecov bot commented Mar 15, 2024

Codecov Report

Merging #3948 (c99ba53) into main (0b9dfb6) will decrease coverage by 14.01%.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3948       +/-   ##
===========================================
- Coverage   73.83%   59.83%   -14.01%     
===========================================
  Files         219      207       -12     
  Lines       15987    15189      -798     
===========================================
- Hits        11804     9088     -2716     
- Misses       3430     5426     +1996     
+ Partials      753      675       -78     

Copy link

This pull request has been marked stale because it has been open for 10 days with no activity

@github-actions github-actions bot added Stale and removed Stale labels Mar 26, 2024
Copy link

This pull request has been marked stale because it has been open for 10 days with no activity

@github-actions github-actions bot added the Stale label Apr 13, 2024
Signed-off-by: Raghav Kaul <raghavkaul+github@google.com>
@raghavkaul raghavkaul marked this pull request as ready for review April 15, 2024 18:41
@raghavkaul raghavkaul requested a review from a team as a code owner April 15, 2024 18:41
@raghavkaul raghavkaul requested review from naveensrinivasan and spencerschrock and removed request for a team April 15, 2024 18:41
Signed-off-by: Raghav Kaul <raghavkaul+github@google.com>
probes/internal/scripts/setup.go Show resolved Hide resolved
probes/internal/scripts/setup.go Outdated Show resolved Hide resolved
probes/internal/scripts/setup.go Outdated Show resolved Hide resolved
Copy link
Contributor

@spencerschrock spencerschrock left a comment

Choose a reason for hiding this comment

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

Please fix the DCO on your most recent commit and address the small registration feedback.

probes/internal/scripts/setup.go Outdated Show resolved Hide resolved
Signed-off-by: Raghav Kaul <raghavkaul+github@google.com>
Signed-off-by: Raghav Kaul <raghavkaul+github@google.com>
@raghavkaul raghavkaul enabled auto-merge (squash) May 8, 2024 17:47
@raghavkaul raghavkaul merged commit 256d5a3 into ossf:main May 8, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants