-
Notifications
You must be signed in to change notification settings - Fork 16
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: implement bldr update --dry
command
#63
Conversation
/approve |
would be nice to see output example in the comment ;) |
Updated description |
/approve |
) | ||
|
||
func gitHubTokenFromEnv() string { | ||
token := os.Getenv("BLDR_GITHUB_TOKEN") |
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.
Why do we need both?
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.
It is just for convenience. BLDR_GITHUB_TOKEN is the bldr-specific token. If it is empty, we try GITHUB_TOKEN, which a common name for many tools that use GitHub API. We don't need any scope for it, so any working token will do.
} | ||
|
||
concurrency := runtime.GOMAXPROCS(-1) | ||
var wg sync.WaitGroup |
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.
Should we use an error group? Do we want to treat a failed update is an 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.
At the moment, all errors are treated as warnings and logged and ignored below. Once more cases are covered, and the distinction between hard errors and warnings is more clear, this code will be updated to use ErrGroup.
Only for GitHub tags and releases for now. Refs siderolabs#48. Signed-off-by: Alexey Palazhchenko <[email protected]>
/lgtm |
Only for GitHub tags and releases for now.
Refs #48.