-
Notifications
You must be signed in to change notification settings - Fork 4.3k
Add plugin: MDX Support #8455
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
base: master
Are you sure you want to change the base?
Add plugin: MDX Support #8455
Conversation
Hello!I found the following issues in your plugin submission Errors: ❌ You did not follow the pull request template. The PR template can be found here This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it. |
|
Updated PR description to follow the template format exactly. Ready for validation. |
|
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1] [1] [1][2][3][4][5] OptionalDo NOT open a new PR for re-validation. |
|
Thank you for the review! I've addressed all the feedback: Sentence case fixes:
Async function fixes:
All changes have been pushed and the build passes successfully. |
|
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1][2] OptionalDo NOT open a new PR for re-validation. |
|
Fixed the remaining sentence case issues - changed 'MDX' back to lowercase 'mdx' in descriptions to match the existing style throughout the settings file. HTML remains uppercase as it's a formal acronym. |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/ddunnock/mdx-support
Release Checklist
main.jsmanifest.jsonstyles.css(optional)v)idin mymanifest.jsonmatches theidin thecommunity-plugins.jsonfile.I have given proper attribution to these other projects in my
README.md.