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

Cleanup: Delete frontmatter keys 'aliases' and 'weight' #118

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

zuiderkwast
Copy link
Contributor

I've cleared with @stockholmux that these frontmatter metadata keys are not useful to keep. They are leftovers from Redis website toolchain.

Deleted using

cd topics
perl -i -ne '$on = 1 if !$on && /^aliases: *\[/; print if !$on; $on = 0 if $on && /^ *\]/' *.md
perl -i -ne '$on = 0 if $on && !/^ /; $on = 1 if !$on && /^aliases:/; print if !$on' *.md
perl -i -ne 'print unless /^weight: *\d+/' *.md

Deleted using

    cd topics
    perl -i -ne '$on = 1 if !$on && /^aliases: *\[/; print if !$on; $on = 0 if $on && /^ *\]/' *.md
    perl -i -ne '$on = 0 if $on && !/^ /; $on = 1 if !$on && /^aliases:/; print if !$on' *.md
    perl -i -ne 'print unless /^weight: *\d+/' *.md

Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
@zuiderkwast zuiderkwast merged commit b4afbed into valkey-io:main Jun 18, 2024
2 checks passed
@zuiderkwast zuiderkwast deleted the delete-unused-frontmatter branch June 18, 2024 09:27
madolson pushed a commit that referenced this pull request Jun 20, 2024
The changes in #118 pushed in malformed YAML frontmatter to
[`/topics/installation.md`](https://github.com/valkey-io/valkey-doc/blob/main/topics/installation.md).

This is causing an error message here on GitHub and the website build to
choke.

This PR removes the orphaned array items when `aliases:` was removed by
the script.

Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>
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.

1 participant