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 default remark plugin definition to theme #3143

Closed
wants to merge 3 commits into from

Conversation

rogermparent
Copy link
Contributor

@rogermparent rogermparent commented Dec 29, 2021

Extracted from #3133

This PR adds a default remark transformer definition to our Gatsby theme. It's actually the one from cml.dev and isn't used on dvc.org since dvc.org has some special plugins that cml.dev doesn't, but it seems like the CML version is the one we'd use on most sites anyway.

Ideally, this remark definition can be used on most of our sites that will use this theme.

@shcheklein shcheklein temporarily deployed to dvc-org-add-markdown-to-fztwzd December 29, 2021 23:15 Inactive
@@ -0,0 +1 @@
// noop
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just something I noticed in other gatsby plugins/themes that seems to be standard practice if you're not using index.js.

Copy link
Contributor

@julieg18 julieg18 left a comment

Choose a reason for hiding this comment

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

Looks good!

@shcheklein shcheklein added A: website Area: website website: eng-doc DEPRECATED JS engine for /doc labels Dec 31, 2021
@shcheklein
Copy link
Member

@julieg18 @rogermparent why this is not merged still? :)

@rogermparent
Copy link
Contributor Author

why this is not merged still?

Same as #3133, #3170 takes its place.

@rogermparent rogermparent deleted the add-markdown-to-plugin branch January 25, 2022 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: website Area: website website: eng-doc DEPRECATED JS engine for /doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants