Skip to content
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

Adds an admonition for exclusive drive access #1196

Merged
merged 2 commits into from
Apr 25, 2024
Merged

Adds an admonition for exclusive drive access #1196

merged 2 commits into from
Apr 25, 2024

Conversation

djwfyi
Copy link
Collaborator

@djwfyi djwfyi commented Apr 24, 2024

Admonition added as an include, then put in many files throughout the docs.
Where it is included, it's the same admonition and text.

It's worth making sure where I placed the include makes sense.

Closes #1186

Staged:

Here is a link to one place where you can see the include in in each of Linux docs and k8s docs:

Admonition added as an include, then icnluded in many files throughout the docs.

Closes #1186
@djwfyi djwfyi self-assigned this Apr 24, 2024
Copy link
Collaborator

@ravindk89 ravindk89 left a comment

Choose a reason for hiding this comment

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

Lgtm mod all pending feedback

Copy link
Collaborator

@feorlen feorlen left a comment

Choose a reason for hiding this comment

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

💥
left some ideas for wording and placement, if you like

@djwfyi djwfyi merged commit c676da2 into main Apr 25, 2024
@djwfyi djwfyi deleted the backend branch April 25, 2024 18:19
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.

Emphasize that users should not modify data files directly
3 participants