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

Add project: mkdocs-snippets #248

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SaltyAimbOtter
Copy link

What kind of change does this PR introduce?

  • Add a project
  • Update a project
  • Remove a project
  • Add or update a category
  • Change configuration
  • Documentation
  • Other, please describe:

Description:

I added the snippet plugin I wrote a while back for another project. It has a few cool features and I hope more people will reuse it if it is listed here.

Checklist:

  • I have read the CONTRIBUTING guidelines.
  • I have not modified the README.md file. Projects are only supposed to be added or updated within the projects.yaml file since the README.md file is automatically generated.

@SaltyAimbOtter
Copy link
Author

CI fail cause:

 mkdocs-snippets:
     'plugin' label should be present if and only if 'mkdocs_plugin:'    

What's the matter the mkdocs_plugin? It is not documented in the CONTRIBUTING.md

@pawamoy
Copy link
Collaborator

pawamoy commented Aug 18, 2024

@SaltyAimbOtter if you look at other projects above, you'll see they specify a mkdocs_plugin key-value pair. It's the name of the plugin, as specified in mkdocs.yml under the plugins configuration item. Sorry the contribution guide doesn't mention it, I believe it's the original CONTRIBUTING file from the repo we forked, and we didn't update it with our customizations.

@pawamoy pawamoy mentioned this pull request Aug 18, 2024
8 tasks
@pawamoy
Copy link
Collaborator

pawamoy commented Oct 10, 2024

@SaltyAimbOtter still interested in these PRs? 🙂

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.

2 participants