-
Notifications
You must be signed in to change notification settings - Fork 51
maintained actions #2515
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
maintained actions #2515
Conversation
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.
Code Review Skipped
StepSecurity AI-CodeWise is designed to handle a maximum of 10 file changes per pull request. To utilize its capabilities, please create a new pull request containing no more than 10 files
Feedback
We appreciate your feedback in helping us improve the service! To provide feedback, please use emojis on this comment. If you find the comments helpful, give them a 👍. If they aren't useful, kindly express that with a 👎. If you have questions or detailed feedback, please create n GitHub issue in StepSecurity/AI-CodeWise.
|
||
func GetLatestRelease(ownerRepo string) (string, error) { | ||
// Build the URL dynamically and add `/actions` at the end | ||
url := fmt.Sprintf("https://api.github.com/repos/%s/releases/latest", ownerRepo) |
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.
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.
Code Review Skipped
StepSecurity AI-CodeWise is designed to handle a maximum of 10 file changes per pull request. To utilize its capabilities, please create a new pull request containing no more than 10 files
Feedback
We appreciate your feedback in helping us improve the service! To provide feedback, please use emojis on this comment. If you find the comments helpful, give them a 👍. If they aren't useful, kindly express that with a 👎. If you have questions or detailed feedback, please create n GitHub issue in StepSecurity/AI-CodeWise.
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.
Code Review Skipped
StepSecurity AI-CodeWise is designed to handle a maximum of 10 file changes per pull request. To utilize its capabilities, please create a new pull request containing no more than 10 files
Feedback
We appreciate your feedback in helping us improve the service! To provide feedback, please use emojis on this comment. If you find the comments helpful, give them a 👍. If they aren't useful, kindly express that with a 👎. If you have questions or detailed feedback, please create n GitHub issue in StepSecurity/AI-CodeWise.
} | ||
|
||
// LoadMaintainedActions loads the maintained actions from the JSON file | ||
func LoadMaintainedActions() (map[string]string, error) { |
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.
the plan is to get this list from the API now, right?
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.
yes
@shubham-stepsecurity can you please review this PR? |
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.
Code Review Skipped
StepSecurity AI-CodeWise is designed to handle a maximum of 10 file changes per pull request. To utilize its capabilities, please create a new pull request containing no more than 10 files
Feedback
We appreciate your feedback in helping us improve the service! To provide feedback, please use emojis on this comment. If you find the comments helpful, give them a 👍. If they aren't useful, kindly express that with a 👎. If you have questions or detailed feedback, please create n GitHub issue in StepSecurity/AI-CodeWise.
ignoreMissingKBs := false | ||
enableLogging := false | ||
exemptedActions, pinToImmutable := []string{}, false | ||
enableLogging := false |
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.
indentation
enableLogging := false | ||
exemptedActions, pinToImmutable := []string{}, false | ||
enableLogging := false | ||
exemptedActions, pinToImmutable, customerMaintainedActions := []string{}, false, map[string]string{} |
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.
customerMaintainedActions should not use word customer
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.
Code Review Skipped
StepSecurity AI-CodeWise is designed to handle a maximum of 10 file changes per pull request. To utilize its capabilities, please create a new pull request containing no more than 10 files
Feedback
We appreciate your feedback in helping us improve the service! To provide feedback, please use emojis on this comment. If you find the comments helpful, give them a 👍. If they aren't useful, kindly express that with a 👎. If you have questions or detailed feedback, please create n GitHub issue in StepSecurity/AI-CodeWise.
please share your thoughts