Skip to content

chore: some improvements in bladectl sub-command handling, error logging, and CI #51

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

Merged
merged 1 commit into from
May 3, 2025

Conversation

cedi
Copy link
Member

@cedi cedi commented Apr 27, 2025

feat(bladectl): implement command structure for managing compute-blade features
feat(bladectl): improve error handling in identify command for better user feedback
feat(goreleaser): add versioning information to builds for better traceability
feat(agent): expose version, commit, and date information in logs for better tracking
chore(ci): update Go setup action to v5 and simplify caching configuration for improved performance
chore(release): update Go setup action to v5 and simplify caching configuration for improved performance
chore(go.mod): update some dependencies to latest versions for improved stability and features


This PR is to clean up the bladectl command structure a little bit and align it closer to the intended format of cobra with <binary> <verb> <noun> which is similar to kubectl

Old New
bladectl fan set-percent 50 bladectl set fan --percent 50
bladectl identify --wait bladectl set confirm --wait

Additionally, this PR improves on the error handling ever so slightly by slowly introducing (humane-errors-go](https://github.com/SierraSoftworks/humane-errors-go) which not only provides an error message, but also adds an actionable advice to the user when showing the error.

Lastly, this PR is modifying the CI pipelines by upgrading to actions/setup-go@v5 which has caching included, simplifying the GitHub Action workflows.

…ing, and CI

chore(ci): update Go setup action to v5 and simplify caching configuration for improved performance
chore(release): update Go setup action to v5 and simplify caching configuration for improved performance
fix(.gitignore): add .idea directory to ignore list to prevent IDE files from being tracked
feat(goreleaser): add versioning information to builds for better traceability
feat(agent): expose version, commit, and date information in logs for better tracking
feat(bladectl): implement command structure for managing compute-blade features
fix(bladectl): improve error handling in identify command for better user feedback
chore(go.mod): update dependencies to latest versions for improved stability and features
Copy link
Member

@xvzf xvzf left a comment

Choose a reason for hiding this comment

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

Really like this, thanks a ton!

@xvzf xvzf merged commit ec6229a into compute-blade-community:main May 3, 2025
4 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.

2 participants