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

feat: restructure; update naming pattern and weight; fix inner-link #137

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

dextermallo
Copy link
Contributor

@dextermallo dextermallo commented Aug 5, 2024

Proposed changes

Implement #133

Further comments

(Some headers of page may be incorrect or be duplicated - I don't want to change them at the moment, just want to show the effects of re-order)

@fzipi fzipi requested a review from a team October 11, 2024 01:21
@fzipi
Copy link
Member

fzipi commented Oct 16, 2024

Hi @dextermallo ! Can you rebase your changes?

@dextermallo
Copy link
Contributor Author

dextermallo commented Oct 17, 2024

Hi @dextermallo ! Can you rebase your changes?

Done in 0f7a705. Also fix an image-link error :)

@@ -1,6 +1,6 @@
---
title: Plugin Mechanism
weight: 40
title: What is Plugin
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
title: What is Plugin
title: Plugins

Copy link
Contributor Author

@dextermallo dextermallo Nov 13, 2024

Choose a reason for hiding this comment

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

Fixed in 53a0433

title: Miscellaneous
weight: 70
pre: "<b>7. </b>"
title: Advanced Usages
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
title: Advanced Usages
title: Advanced Topics

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 53a0433

@@ -1,8 +1,8 @@
---
title: Known Issues
weight: 10
title: 7. Known Issues
Copy link
Contributor

Choose a reason for hiding this comment

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

Use <pre>...?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 53a0433

renovate bot and others added 5 commits November 13, 2024 08:26
…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]>
@dextermallo
Copy link
Contributor Author

dextermallo commented Nov 13, 2024

Hmm not sure what bumped into my PR; every time I rebase, the PR reverts some of my previous changes.
Perhaps I should wait until folks make a decision to move forward and then make changes one time. Otherwise I have to keep updating it... :(

(The conflicts I saw in the PR should not be existed since the files are renamed and moved)

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.

4 participants