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

Go: Implement Scan #3378

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Go: Implement Scan #3378

wants to merge 15 commits into from
+172 −0

Conversation

EdricCua
Copy link
Collaborator

@EdricCua EdricCua commented Mar 17, 2025 •
edited
Loading

Issue link

This Pull Request is linked to issue (URL): #3401

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one issue.
  • Commit message has a detailed description of what changed and why.
  • Tests are added or updated.
  • CHANGELOG.md and documentation files are updated.
  • Destination branch is correct - main or release
  • Create merge commit if merging release branch into main, squash otherwise.

EdricCua added 4 commits March 17, 2025 16:33
@EdricCua
Implemt scan command
e934cf8 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
Implemt scan command
e806ecc 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
Implemt scan command
474bf2a 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
Implemt scan command
5e40ac8 
Signed-off-by: EdricCua <[email protected]>
@EdricCua EdricCua requested a review from a team as a code owner March 17, 2025 22:50
@EdricCua EdricCua added the go golang wrapper label Mar 18, 2025
EdricCua added 2 commits March 18, 2025 10:24
@EdricCua
Implemt scan command
384751c 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
Implemt scan command
3d60bc2 
Signed-off-by: EdricCua <[email protected]>
Yury-Fridlyand
Yury-Fridlyand reviewed Mar 18, 2025
View reviewed changes
Copy link
Collaborator

@Yury-Fridlyand Yury-Fridlyand left a comment

Choose a reason for hiding this comment

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

Please add changelog and fix PR description (needs a link to an issue)

EdricCua added 6 commits March 20, 2025 02:32
@EdricCua
fix examples
d280ffc 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
update from main
177e6dd 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
fix example
4620c85 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
fix example
db586b8 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
fix example
8e56e7e 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
fix example
36f7bfd 
Signed-off-by: EdricCua <[email protected]>
Yury-Fridlyand
Yury-Fridlyand reviewed Mar 20, 2025
View reviewed changes
@Yury-Fridlyand Yury-Fridlyand mentioned this pull request Mar 20, 2025
Open
3 tasks
@EdricCua
update from main
7ef1356 
Signed-off-by: EdricCua <[email protected]>
EdricCua added 2 commits March 21, 2025 02:17
@EdricCua
fix review comment
fdf6788 
Signed-off-by: EdricCua <[email protected]>
@EdricCua
update from main
d543d1a 
Signed-off-by: EdricCua <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yury-Fridlyand Yury-Fridlyand Yury-Fridlyand approved these changes

@jbrinkman jbrinkman Awaiting requested review from jbrinkman

Assignees
No one assigned
Labels
go golang wrapper
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@EdricCua @Yury-Fridlyand