Skip to content

Avoid CPU:Mem which confuses some markdown parsers. #1023

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Avoid CPU:Mem which confuses some markdown parsers. #1023

wants to merge 7 commits into from
+5 −5

Conversation

@garloff
Copy link
Member

@garloff garloff commented Nov 27, 2025

Cosmetic ...

@garloff garloff self-assigned this Nov 27, 2025
@garloff garloff added bug Something isn't working work in progress Pull requests that are work in progress, do not merge them standards Issues / ADR / pull requests relevant for standardization & certification labels Nov 27, 2025
depressiveRobot
depressiveRobot requested changes Nov 27, 2025
View reviewed changes
Copy link
Contributor

@depressiveRobot depressiveRobot left a comment

Choose a reason for hiding this comment

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

Is this a general problem? As there are many other places like "CPU:Mem" or "1:4". For example see scs-0100-v1-flavor-naming.md

mbuechse
mbuechse reviewed Nov 27, 2025
View reviewed changes
Copy link
Contributor

@mbuechse mbuechse left a comment

Choose a reason for hiding this comment

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

I would apply the same change to https://github.com/SovereignCloudStack/standards/blob/main/Standards/scs-0103-v1-standard-flavors.md?plain=1#L135
I can do it (but I don't know when).

@garloff
Copy link
Member Author

garloff commented Nov 28, 2025 •
edited
Loading

I would apply the same change to https://github.com/SovereignCloudStack/standards/blob/main/Standards/scs-0103-v1-standard-flavors.md?plain=1#L135 I can do it (but I don't know when).

True, thanks for spotting! Done in @b105c8d.

mbuechse
mbuechse approved these changes Nov 28, 2025
View reviewed changes
Copy link
Contributor

@mbuechse mbuechse left a comment

Choose a reason for hiding this comment

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

Docusaurus is good for surprises. Thanks for resolving this!

@garloff
Copy link
Member Author

garloff commented Dec 2, 2025

@depressiveRobot: Did I address your concerns? If so, please re-review (and approve).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mbuechse mbuechse mbuechse approved these changes

@fkr fkr Awaiting requested review from fkr fkr is a code owner

@depressiveRobot depressiveRobot Awaiting requested review from depressiveRobot depressiveRobot is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

@garloff garloff

Labels

bug Something isn't working standards Issues / ADR / pull requests relevant for standardization & certification work in progress Pull requests that are work in progress, do not merge them

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@garloff @depressiveRobot @mbuechse