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 language for .remarkrc, .remarkignore to VSC #78

Merged
merged 2 commits into from
Dec 23, 2021
Merged

Add language for .remarkrc, .remarkignore to VSC #78

merged 2 commits into from
Dec 23, 2021

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

These languages can’t be detected by VSCode by default.

@codecov-commenter

This comment has been minimized.

Comment on lines +220 to +233
"languages": [
{
"id": "ignore",
"extensions": [
".remarkignore"
]
},
{
"id": "json",
"filenames": [
".remarkrc"
]
}
]
Copy link
Member

Choose a reason for hiding this comment

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

This is new-ish to me.
Does this only impact people cloning this repo?
Or does this somehow tell vscode how to highlight these files if remark is installed?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is part of the plugin manifest, not local development settings.

Or does this somehow tell vscode how to highlight these files if remark is installed?

☝️ This exactly

I basically copied it from https://github.com/microsoft/vscode-eslint/blob/main/package.json#L490-L503

Copy link
Member

Choose a reason for hiding this comment

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

Interpreting the :+1: by Christian as an ✅ review

@wooorm wooorm changed the title Define languages for .remarkrc and .remarkignore Add language for .remarkrc, .remarkignore to VSC Dec 23, 2021
@wooorm wooorm merged commit 0f91d78 into remarkjs:main Dec 23, 2021
@wooorm wooorm added the 💪 phase/solved Post is done label Dec 23, 2021
@remcohaszing remcohaszing deleted the define-config-languages branch December 27, 2021 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💪 phase/solved Post is done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants