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 missing type to typescript .d.ts #516

Merged
merged 2 commits into from
Jul 22, 2020
Merged

Conversation

CNLHC
Copy link
Contributor

@CNLHC CNLHC commented Jul 19, 2020

RemarkParser does have blockTokenizers but it's missing here.

This PR trys to add missing type to index.d.ts.

`RemarkParser` does have `blockTokenizers` but it's missing here.
Copy link
Member

@ChristianMurphy ChristianMurphy left a comment

Choose a reason for hiding this comment

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

Thanks @CNLHC!
Having the block tokenizers is a good addition.
I added a few comments below:

packages/remark-parse/types/index.d.ts Outdated Show resolved Hide resolved
packages/remark-parse/types/index.d.ts Outdated Show resolved Hide resolved
@ChristianMurphy ChristianMurphy added ☂️ area/types This affects typings 🐛 type/bug This is a problem 👶 semver/patch This is a backwards-compatible fix labels Jul 19, 2020
@CNLHC
Copy link
Contributor Author

CNLHC commented Jul 21, 2020

Done. \cc @ChristianMurphy maybe you can merge it now.

@ChristianMurphy ChristianMurphy merged commit 19a27c3 into remarkjs:main Jul 22, 2020
@justinbhopper
Copy link

@ChristianMurphy Will this be released released as a stable version for 8.0?

@ChristianMurphy
Copy link
Member

ChristianMurphy commented Oct 9, 2020

I'm not sure, I know v9 alphas are now in progress.

A member of @remarkjs/releasers may be able to comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☂️ area/types This affects typings 👶 semver/patch This is a backwards-compatible fix 🐛 type/bug This is a problem
Development

Successfully merging this pull request may close these issues.

None yet

4 participants