-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: restructure; update naming pattern and weight; fix inner-link #137
base: main
Are you sure you want to change the base?
feat: restructure; update naming pattern and weight; fix inner-link #137
Conversation
Hi @dextermallo ! Can you rebase your changes? |
419494c
to
0f7a705
Compare
Done in 0f7a705. Also fix an image-link error :) |
@@ -1,6 +1,6 @@ | |||
--- | |||
title: Plugin Mechanism | |||
weight: 40 | |||
title: What is Plugin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title: What is Plugin | |
title: Plugins |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 53a0433
content/5-advanced-usages/_index.md
Outdated
title: Miscellaneous | ||
weight: 70 | ||
pre: "<b>7. </b>" | ||
title: Advanced Usages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title: Advanced Usages | |
title: Advanced Topics |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 53a0433
content/7-known-issues/_index.md
Outdated
@@ -1,8 +1,8 @@ | |||
--- | |||
title: Known Issues | |||
weight: 10 | |||
title: 7. Known Issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use <pre>...
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 53a0433
…les (coreruleset#146) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…set#140) * chore(deps): pin dependencies in .github/workflows/test.yml * fix: use user agent that isn't blocked by drupal / sourceforge --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Max Leske <[email protected]>
Hmm not sure what bumped into my PR; every time I rebase, the PR reverts some of my previous changes. (The conflicts I saw in the PR should not be existed since the files are renamed and moved) |
Proposed changes
Implement #133
Further comments