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

refactor(gatsby-transformer-remark): Refactors out Bluebird usage in transformer-remark #29638

Merged
merged 3 commits into from
Mar 3, 2021

Conversation

lourd
Copy link
Contributor

@lourd lourd commented Feb 21, 2021

Just some minor cleanup, noticed the usage of Bluebird's Promise.each was unnecessary when reading through the source.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Feb 21, 2021
@ascorbic ascorbic changed the title Refactors out Bluebird usage in transformer-remark refactor(gatsby-transformer-remark): Refactors out Bluebird usage in transformer-remark Feb 25, 2021
@ascorbic
Copy link
Contributor

Hi. Thanks for this. This is failing tests right now. I think this is because of an error in the code. I've added a suggestion. Could you take a look and see if it fixes it? You can run yarn jest in the root of the monorepo to check.

@ascorbic ascorbic added topic: remark/mdx Related to Markdown, remark & MDX ecosystem and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Feb 25, 2021
@lourd
Copy link
Contributor Author

lourd commented Feb 25, 2021

Updated with your suggestion and merged with master. I don't think the still-failing tests are related to this change?

Copy link
Contributor

@wardpeet wardpeet left a comment

Choose a reason for hiding this comment

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

Thanks!

@wardpeet wardpeet merged commit 7193303 into gatsbyjs:master Mar 3, 2021
pragmaticpat pushed a commit to pragmaticpat/gatsby that referenced this pull request Apr 28, 2022
…transformer-remark (gatsbyjs#29638)

Co-authored-by: Ward Peeters <ward@coding-tech.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: remark/mdx Related to Markdown, remark & MDX ecosystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants