Closed smuu closed 3 months ago
This update primarily involves renaming the SetCommand
method to SetStartCommand
across various components, enhancing clarity regarding its intended purpose. The changes reflect a semantic shift towards defining commands explicitly for starting processes, aligning with a broader refactoring effort across the codebase. Although the logic and control flow remain intact, these modifications improve readability and maintainability.
Files | Change Summary |
---|---|
README.md , e2e/basic/*.go , e2e/netshaper/netshaper_test.go , e2e/system/*.go |
Replaced SetCommand with SetStartCommand , enhancing clarity related to command execution for instances. |
pkg/instance/build.go , pkg/instance/errors.go , pkg/knuu/*.go , pkg/sidecars/*.go |
Renamed methods related to command and capability handling for specificity, reflecting a focus on Kubernetes. |
pkg/knuu/instance_old.go |
Updated deprecated method names to align with new Kubernetes-specific terminology, maintaining existing logic. |
sequenceDiagram
participant User
participant Instance
User->>Instance: SetStartCommand()
Instance-->>User: Command set confirmation
Objective | Addressed | Explanation |
---|---|---|
Rename functions for clarity (#38) | ✅ |
🐇 In the land of code, changes take flight,
With clearer commands, we set things just right!
FromSetCommand
toSetStart
, what a delight,
Our functions now sparkle, oh what a sight!
With each little tweak, we hop with glee,
Improving our code for you and for me! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Overview
Closes https://github.com/celestiaorg/knuu/issues/38
Summary by CodeRabbit
New Features
SetStartCommand
, clarifying command initialization across various components.Bug Fixes
Documentation
Chores