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 loading remark plugins from tsconfig.json #383

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

remcohaszing
Copy link
Member

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

@mdx-js/language-service now exposes a function
resolveRemarkPlugins, which resolves remark plugins in a environment agnostic manner.

The language server implements Node.js specific logic to resolve this from tsconfig.json.

`@mdx-js/language-service` now exposes a function
`resolveRemarkPlugins`, which resolves remark plugins in a environment
agnostic manner.

The language server implements Node.js specific logic to resolve this
from `tsconfig.json`.
@remcohaszing remcohaszing added 🦋 type/enhancement This is great to have 🗄 area/interface This affects the public interface 🏡 area/internal This affects the hidden internals 👶 semver/patch This is a backwards-compatible fix 👍 phase/yes Post is accepted and can be worked on labels Dec 27, 2023
Copy link

changeset-bot bot commented Dec 27, 2023

🦋 Changeset detected

Latest commit: a748c43

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

This PR includes changesets to release 2 packages
Name Type
@mdx-js/language-service Patch
@mdx-js/language-server Patch

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

Copy link
Contributor

Hi! This is accepted and can go somewhere!

Team: please review this PR and use the area/* (to describe the scope of the change), platform/* (if this is related to a specific one), and semver/* and type/* labels to annotate this.

@remcohaszing remcohaszing merged commit 135f633 into main Dec 27, 2023
14 checks passed
@remcohaszing remcohaszing deleted the refactor-plugin-loading branch December 27, 2023 15:23
Copy link
Contributor

Hi! This was closed. Team: If this was merged, please describe when this is likely to be released. Otherwise, please add one of the no/* labels.

@github-actions github-actions bot mentioned this pull request Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗄 area/interface This affects the public interface 🏡 area/internal This affects the hidden internals 👍 phase/yes Post is accepted and can be worked on 👶 semver/patch This is a backwards-compatible fix 🦋 type/enhancement This is great to have
Development

Successfully merging this pull request may close these issues.

None yet

1 participant