Skip to content

devices: reformat switch - #275

Merged
thaJeztah merged 1 commit into
moby:mainfrom
thaJeztah:devices_format
Oct 28, 2026
Merged

thaJeztah merged 1 commit into
moby:mainfrom
thaJeztah:devices_format

Conversation

@thaJeztah

@thaJeztah thaJeztah commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Wrap the cases, so that the corresponding comments can be put next to the item they relate to.

updates 6b651fb

@thaJeztah

Copy link
Copy Markdown
Member Author

Had this in a local branch, so let's open it as a PR 😅

Comment thread devices/device_unix.go Outdated
Comment on lines +102 to +104
".lxc", ".lxd-mounts", // addresses https://github.com/lxc/lxd/issues/2825
".udev", // addresses https://github.com/opencontainers/runc/issues/2093
".incus-mounts": // addresses https://github.com/moby/sys/issues/260

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'd drop the word "addresses", a bare link is fine.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, I was debating as well; considered see XXX, but indeed just the URL would work.

I'll update and bring this one in.

@kolyshkin kolyshkin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm (and a nit -- feel free to either implement or ignore)

Wrap the cases, so that the corresponding comments can be put
next to the item they relate to.

updates 6b651fb

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
thaJeztah merged commit 944ed87 into moby:main Oct 28, 2026
25 checks passed
@thaJeztah
thaJeztah deleted the devices_format branch October 28, 2026 09:06
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