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

Improve export name extraction for shortcode generation #1160

Merged
merged 6 commits into from
Jul 17, 2020
Merged

Conversation

johno
Copy link
Member

@johno johno commented Jul 17, 2020

Many different types of export syntax weren't supported,
primarily destructuring for objects and arrays. This adds
export extraction to the import name extraction step so
we can more accurately determine what shortcodes to
generate.

We will eventually want to use something different for
extraction (#1152), but for now let's just make it right before
we make it fast.

Many different types of export syntax wasn't supported,
primarily destructuring for objects and arrays. This adds
export extraction to the import name extraction step so
we can more accurately determine what shortcodes to
generate.
@johno johno added the 💎 v2 Issues related to v2 label Jul 17, 2020
@vercel
Copy link

vercel bot commented Jul 17, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/mdx/mdx/qwkwpf4iv
✅ Preview: https://mdx-git-export-handling.mdx.vercel.app

@vercel vercel bot temporarily deployed to Preview July 17, 2020 20:13 Inactive
Copy link

@laurieontech laurieontech left a comment

Choose a reason for hiding this comment

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

Any idea why the tests are failing? I could be missing something but I think my comment and the other default value [] will never hit the return statement.

@vercel vercel bot temporarily deployed to Preview July 17, 2020 20:14 Inactive
@vercel vercel bot temporarily deployed to Preview July 17, 2020 20:18 Inactive
@vercel vercel bot temporarily deployed to Preview July 17, 2020 20:20 Inactive
@vercel vercel bot temporarily deployed to Preview July 17, 2020 20:23 Inactive
@johno johno merged commit db93304 into next Jul 17, 2020
@johno johno deleted the export-handling branch July 17, 2020 20:24
Copy link

@laurieontech laurieontech left a comment

Choose a reason for hiding this comment

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

Looks reasonable

@johno johno mentioned this pull request Jul 22, 2020
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💎 v2 Issues related to v2
Development

Successfully merging this pull request may close these issues.

2 participants