fix changed not saved to the file

This commit is contained in:
Maksim Smirnov
2025-08-23 09:49:33 +02:00
parent 920de1c512
commit 072582d9cb

View File

@@ -50,12 +50,18 @@ func FixCommand(c *cli.Context) error {
// Run line checkers and fixers. // Run line checkers and fixers.
lines := SplitIntoLines(fileContent) lines := SplitIntoLines(fileContent)
lineNo := 1 lineNo := 1
for _, line := range lines { for i, line := range lines {
for ruleName, ruleValue := range rules { for ruleName, ruleValue := range rules {
if lineChecker, ok := lineCheckers[ruleName]; ok { if lineChecker, ok := lineCheckers[ruleName]; ok {
result := lineChecker(ruleValue, line) result := lineChecker(ruleValue, line)
if !result.isOk { if !result.isOk {
fmt.Println(f + ": line " + strconv.Itoa(lineNo) + ": " + ruleName + ": " + result.messageIfNotOk) if result.fixer != nil {
lines[i] = result.fixer(ruleValue, line)
hasChanged = true
fmt.Println(f + ": line " + strconv.Itoa(lineNo) + ": " + ruleName + ": fixed")
} else {
fmt.Println(f + ": line " + strconv.Itoa(lineNo) + ": " + ruleName + ": " + result.messageIfNotOk)
}
// Don't show more than 1 error per line. // Don't show more than 1 error per line.
break break
} }
@@ -63,14 +69,19 @@ func FixCommand(c *cli.Context) error {
} }
lineNo++ lineNo++
} }
// Rejoin the lines if any were changed
if hasChanged {
fileContent = strings.Join(lines, "\n")
}
if hasChanged { if hasChanged {
fileHandler, err := os.Open(f) err := os.WriteFile(f, []byte(fileContent), 0644)
if err != nil { if err != nil {
fmt.Println("Could not write to " + f) fmt.Println("Could not write to " + f + ": " + err.Error())
} else {
fmt.Println("Wrote to " + f)
} }
fileHandler.WriteString(fileContent)
fmt.Println("Wrote to " + f)
} }
} }