Skip to content

internal/exec/stages/files/units: fix formating error cought downstream#2210

Open
prestist wants to merge 1 commit intocoreos:mainfrom
prestist:update-for-go-version
Open

internal/exec/stages/files/units: fix formating error cought downstream#2210
prestist wants to merge 1 commit intocoreos:mainfrom
prestist:update-for-go-version

Conversation

@prestist
Copy link
Collaborator

The systemdVersion variable is an int and should use %d format specifier, not %q (which is for strings).

The systemdVersion variable is an int and should use %d format
specifier, not %q (which is for strings).
@prestist prestist added the skip-notes This PR does not need release notes label Mar 13, 2026
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request correctly fixes a formatting error in a warning message by changing the format specifier for an integer from %q to %d. I've also added a suggestion to improve maintainability by replacing a magic number with a constant.

@prestist prestist force-pushed the update-for-go-version branch from 606a6dc to cdb7935 Compare March 13, 2026 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-notes This PR does not need release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant