-
Notifications
You must be signed in to change notification settings - Fork 508
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
✨ Miscellaneous refactors to ease downstream consumption #1645
Conversation
0e0ac01
to
29f9cd8
Compare
Integration tests success for |
Codecov Report
@@ Coverage Diff @@
## main #1645 +/- ##
==========================================
+ Coverage 58.50% 60.58% +2.08%
==========================================
Files 58 58
Lines 5801 5907 +106
==========================================
+ Hits 3394 3579 +185
+ Misses 2170 2085 -85
- Partials 237 243 +6 |
Integration tests success for |
29f9cd8
to
a14d103
Compare
Integration tests success for |
a14d103
to
4e4795b
Compare
Integration tests success for |
Integration tests success for |
fe71973
to
aa712bc
Compare
Integration tests success for |
e00aaeb
to
e3f9f95
Compare
Integration tests success for |
Integration tests success for |
e3f9f95
to
2373cbb
Compare
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
e9d8801
to
25bba32
Compare
Integration tests success for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I am OK with this change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Integration tests success for |
NewLogger
constructor forDetailLogger
implNewRunner
constructor for Runneroptions
packagegithubrepo
packagepolicy
packageformat
packageSigned-off-by: Stephen Augustus [email protected]
Tested against ossf/allstar#114.
cc: @jeffmendoza
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information: