-
Notifications
You must be signed in to change notification settings - Fork 418
Open
Labels
backendIssues related to the backendIssues related to the backendgood first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.qualityRelated to improving the quality of the appRelated to improving the quality of the app
Description
This function needs to be shorter to reduce the complexity of it, and make it easier to understand.
Currently the code turns off the linter for this issue with "//nolint:funlen".
- refactor the code to make the function short and easier to understand
- See the contributing guide on how to get started.
- See headlamp/backend/cmd/headlamp.go file.
- To run the linter,
make backend-lint
- How to run tests:
make backend-test
To pick up a draggable item, press the space bar. While dragging, use the arrow keys to move the item. Press space again to drop the item in its new position, or press escape to cancel.
Related umbrella backend refactoring clean up issue #855 (other pieces of code have the linter disabled and could also be fixed up, feel free to refactor a different function.) If you've already refactored one function, please leave the rest for others.
Metadata
Metadata
Assignees
Labels
backendIssues related to the backendIssues related to the backendgood first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.qualityRelated to improving the quality of the appRelated to improving the quality of the app
Type
Projects
Status
Queued
Activity
goyalg325 commentedon May 18, 2025
Hi @illume,
Could you please assign this issue to me? I'd like to work on it.
Thank you!
AyushSriv06 commentedon May 19, 2025
@goyalg325 are you still working on the issue?
goyalg325 commentedon May 19, 2025
Yes!
Fix kubernetes-sigs#3273: Refactor createHeadlampHandler to reduce co…
Darshit42 commentedon May 27, 2025
is this issue still open?
SinghaAnirban005 commentedon May 29, 2025
@goyalg325 if you aren't currently working on the issue
I would love to go ahead with this
Thank you
7908837174 commentedon May 29, 2025
backend: Refactor createHeadlampHandler to reduce complexity
8 remaining items