-
Notifications
You must be signed in to change notification settings - Fork 239
New Linter integration #2007
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
New Linter integration #2007
Conversation
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.
Pull Request Overview
This PR updates code to comply with new linter rules by refactoring code formatting and style issues. The changes focus on improving readability through proper line breaks, function argument formatting, and conditional statement simplification.
- Function call arguments are now split across multiple lines when they exceed length limits
- Long conditional statements are broken into multiple lines for better readability
- Complex log statements are formatted with proper line breaks
Reviewed Changes
Copilot reviewed 68 out of 70 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/health-monitor/monitor/network_profiler/network_monitor.go | Split long log error statements across multiple lines |
| tools/health-monitor/monitor/file_cache/cache_monitor.go | Reformatted cache consumption calculations with proper line breaks |
| tools/health-monitor/monitor/cpu_mem_profiler/cpu_mem_monitor.go | Split complex log statements and conditional expressions |
| tools/health-monitor/monitor/blobfuse_stats/stats_reader.go | Replaced WriteString with fmt.Fprintf for better formatting |
| tools/health-monitor/main.go | Reformatted flag initialization and debug logging statements |
| tools/health-monitor/internal/stats_export.go | Converted if-else chains to switch statements and formatted function calls |
| main.go | Removed duplicate import statement |
| internal/stats_manager/stats_manager.go | Updated WriteString calls to fmt.Fprintf |
| internal/pipeline.go | Simplified conditional logic and formatted error messages |
| go.mod | Updated Go version from 1.24.4 to 1.25.1 and dependency versions |
| component/xload/*.go | Extensive formatting of log statements and function calls across multiple files |
| component/loopback/loopback_fs.go | Formatted function signatures and string formatting calls |
| component/libfuse/*.go | Split long function signatures and reformatted conditional statements |
| component/file_cache/*.go | Converted if-else chains to switch statements and formatted long statements |
| component/entry_cache/entry_cache.go | Split function signatures and log statements |
| component/custom/custom.go | Formatted error messages across multiple lines |
| component/block_cache/*.go | Extensive reformatting of function calls and conditional statements |
| component/azstorage/*.go | Formatted function calls, conditional statements, and error handling |
| component/attr_cache/attr_cache.go | Simplified conditional logic and formatted function signatures |
| common/*.go | Various formatting improvements including function signatures and variable declarations |
| cmd/*.go | Formatted error messages and function calls |
Updating code as per new linter rules.