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

New Markdown API #2862

Merged
merged 7 commits into from
Mar 29, 2022
Merged

New Markdown API #2862

merged 7 commits into from
Mar 29, 2022

Conversation

FredKSchott
Copy link
Member

@FredKSchott FredKSchott commented Mar 23, 2022

Changes

Testing

  • Covered by existing tests

Docs

  • TODO, still needed

@changeset-bot
Copy link

changeset-bot bot commented Mar 23, 2022

🦋 Changeset detected

Latest commit: 725c367

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

This PR includes changesets to release 1 package
Name Type
astro Minor

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) pkg: example Related to an example package (scope) labels Mar 23, 2022
@github-actions github-actions bot added the test label Mar 23, 2022
Copy link
Contributor

@bholmesdev bholmesdev left a comment

Choose a reason for hiding this comment

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

Small comments but looks globalicious!

examples/blog-multiple-authors/src/pages/index.astro Outdated Show resolved Hide resolved
examples/blog/astro.config.mjs Outdated Show resolved Hide resolved
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 good! A few comments and suggestions.

packages/astro/src/vite-plugin-markdown/index.ts Outdated Show resolved Hide resolved
packages/astro/src/@types/astro.ts Outdated Show resolved Hide resolved
@FredKSchott FredKSchott marked this pull request as draft March 25, 2022 17:50
@FredKSchott
Copy link
Member Author

Marking as a draft so that no one thinks to merge this for v0.25 :) Will re-mark as ready to merge starting on Monday!

@FredKSchott FredKSchott marked this pull request as ready for review March 27, 2022 04:57
@@ -1,6 +1,6 @@
/// <reference types="vite/client" />

type Astro = import('./dist/types/@types/astro').AstroGlobal;
type Astro = import('astro').AstroGlobal;
Copy link
Member Author

Choose a reason for hiding this comment

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

confirmed this continues to work in installed dependencies, but now it also works in our monorepo

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! Excited to land this.

packages/astro/src/@types/astro.ts Outdated Show resolved Hide resolved
@FredKSchott FredKSchott merged commit 4299ab3 into main Mar 29, 2022
@FredKSchott FredKSchott deleted the deferred-markdown branch March 29, 2022 00:16
@github-actions github-actions bot mentioned this pull request Mar 29, 2022
SiriousHunter pushed a commit to SiriousHunter/astro that referenced this pull request Feb 3, 2023
* Implement new markdown plugin with deferred markdown rendering

* feat: switch from `getContent()` fn to `<Content />` API

* update types

* Update packages/astro/src/@types/astro.ts

Co-authored-by: Nate Moore <[email protected]>

* update types

* Create forty-coins-attend.md

Co-authored-by: Nate Moore <[email protected]>
Co-authored-by: Nate Moore <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: astro Related to the core `astro` package (scope) pkg: example Related to an example package (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants