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

Remove Astro-flavored Markdown from @astrojs/markdown-remark #5785

Merged
merged 11 commits into from
Jan 9, 2023

Conversation

delucis
Copy link
Member

@delucis delucis commented Jan 6, 2023

Changes

Cleans up @astrojs/markdown-remark removing syntax tooling that is no longer needed now that we are not supporting Astro-flavored Markdown.

  • Removes our custom remark/rehype plugins that powered AFMD
  • Removes dependencies for those plugins that aren’t used elsewhere
  • Updates our headings plugin to remove some AFMD-specific code
  • Removes the astroFlavoredMd option from the renderMarkdown method

Testing

I’ve removed AFMD-specific tests.

Docs

We’re already documenting AFMD’s demise, so no additional docs associated with this PR.

@changeset-bot
Copy link

changeset-bot bot commented Jan 6, 2023

🦋 Changeset detected

Latest commit: e4fcbc0

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added pkg: astro Related to the core `astro` package (scope) feat: markdown Related to Markdown (scope) labels Jan 6, 2023
@delucis delucis changed the title [WIP] Remove remaining Astro-flavored Markdown code Remove remaining Astro-flavored Markdown code Jan 6, 2023
@delucis delucis marked this pull request as ready for review January 6, 2023 18:50
@delucis delucis changed the title Remove remaining Astro-flavored Markdown code Remove Astro-flavored Markdown code from @astrojs/markdown-remark Jan 6, 2023
@delucis delucis self-assigned this Jan 6, 2023
Copy link
Member

@natemoo-re natemoo-re left a comment

Choose a reason for hiding this comment

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

Looks great! Needs a major changeset?

@delucis
Copy link
Member Author

delucis commented Jan 6, 2023

Got one ready to push but didn’t want to disturb the test run 😅

@delucis
Copy link
Member Author

delucis commented Jan 6, 2023

Heading out for the day but if it’s helpful to merge this at any point this afternoon, please do!

@delucis delucis changed the title Remove Astro-flavored Markdown code from @astrojs/markdown-remark Remove Astro-flavored Markdown from @astrojs/markdown-remark Jan 9, 2023
@delucis delucis merged commit 16107b6 into main Jan 9, 2023
@delucis delucis deleted the chris/goodbye-afmd branch January 9, 2023 09:23
delucis added a commit that referenced this pull request Jan 9, 2023
Follow up to #5785. Missed one dependency that is no longer used and can be removed.
delucis added a commit that referenced this pull request Jan 9, 2023
* [markdown-remark] Remove unused dependency

Follow up to #5785. Missed one dependency that is no longer used and can be removed.

* [markdown-remark] Remove unused dependency

* Lockfile fix

* Update lockfile [again]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: markdown Related to Markdown (scope) pkg: astro Related to the core `astro` package (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants