Skip to content

Change %q in logs and errors to %#q [Part3]#5029

Open
mie313 wants to merge 1 commit into
lima-vm:masterfrom
mie313:use_hashq_part3
Open

Change %q in logs and errors to %#q [Part3]#5029
mie313 wants to merge 1 commit into
lima-vm:masterfrom
mie313:use_hashq_part3

Conversation

@mie313
Copy link
Copy Markdown
Contributor

@mie313 mie313 commented May 23, 2026

Change %q in logs and errors to %#q and update the codes so that backticks are used everywhere in logs and errors. (all files in the following directories) .

  • pkg/guestagent
  • pkg/guestpatch
  • pkg/hostagent
  • pkg/httpclientutil
  • pkg/identifiers
  • pkg/imgutil
  • pkg/instance
  • pkg/iso9660util
  • pkg/limactlutil
  • pkg/limainfo
  • pkg/limatmpl
  • pkg/limatype

Related issue: #4898
This PR is a continuation of the work from #4909 and #5017 .

@mie313 mie313 force-pushed the use_hashq_part3 branch 3 times, most recently from 99fdc19 to 98548d5 Compare May 25, 2026 09:07
@mie313 mie313 marked this pull request as ready for review May 25, 2026 09:25
homedir := u.Homedir
if homedir == "" {
return fmt.Errorf("homedir is required for user %q", u.Name)
return fmt.Errorf("homedir is required for user %#qq", u.Name)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You have again included a couple of %#qq instances in the PR. How do they happen? Did you create this with AI without checking the output?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Apologies for the inconvenience, but I have never used AI at all for contributing lima.
Since I modified all codes at once and then divided them into a few PRs, typos the same as before might remain.
I will make sure to double-check not to contain any typos before turning a PR ready for review.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed all typos and squashed 🙏

Signed-off-by: mie313 <mie313.tanaka@gmail.com>
@mie313 mie313 force-pushed the use_hashq_part3 branch from 98548d5 to dd5b460 Compare May 25, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants