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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add --check flag to the update command #49

Merged
merged 11 commits into from
Jul 3, 2024

Conversation

hanshal101
Copy link
Contributor

CLOSES #48
FIXES #48

Description

This PR adds the functionality to check for updates before updating them through --check flag.

@rakshitgondwal
Copy link
Member

Hi @hanshal101, Thank you for the PR. Can you please sign off your commits?

@hanshal101 hanshal101 force-pushed the bsf-update-check branch 3 times, most recently from 8bb71f6 to 1543488 Compare July 3, 2024 21:55
@dr-housemd
Copy link
Contributor

also, can you please add yourself to CONTRIBUTORS file as mentioned here?
Just make a dedicated commit to do so
https://github.com/buildsafedev/bsf?tab=readme-ov-file#contributing

Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
@hanshal101 hanshal101 force-pushed the bsf-update-check branch 2 times, most recently from dfdef8e to 26a1222 Compare July 3, 2024 21:56
cmd/update/update.go Outdated Show resolved Hide resolved
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
pkg/update/update.go Outdated Show resolved Hide resolved
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
pkg/update/update.go Outdated Show resolved Hide resolved
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
cmd/update/update.go Outdated Show resolved Hide resolved
cmd/update/update.go Outdated Show resolved Hide resolved
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
pkg/update/update.go Outdated Show resolved Hide resolved
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
pkg/update/update.go Outdated Show resolved Hide resolved
Signed-off-by: hanshal101 <82961842+hanshal101@users.noreply.github.com>
Copy link
Contributor

@dr-housemd dr-housemd left a comment

Choose a reason for hiding this comment

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

LGTM.
Thanks @hanshal101 for this PR ,your patience with the review and quick response!

@dr-housemd dr-housemd merged commit 15a1e4c into buildsafedev:main Jul 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bsf update --check
3 participants