mirror of
https://github.com/awatson1978/meteor-api.git
synced 2026-05-16 14:15:59 -06:00
[GH-ISSUE #80] Overriding default file grammars automatically #64
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/meteor-api#64
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @jacobmischka on GitHub (Apr 5, 2016).
Original GitHub issue: https://github.com/awatson1978/meteor-api/issues/80
Currently, when I install meteor-api it overrides the default grammars for
.jsand.csonand other general filetypes. This is not correct behavior as mentioned in the docs, I did not manually disable the language-javascript package.I'd much rather have to manually select the Meteor syntax when working in meteor than deselect it when I'm not.
@LeNiglo commented on GitHub (Dec 20, 2016):
Months later, it's still the case.
Any chance we get an update on this ?
@awatson1978 commented on GitHub (Dec 20, 2016):
Hi there. The Atom Package Manager account is borked. I've logged requests for someone to reset it, but it may be easier to create a new package. Not much I can do about it.
@jacobmischka commented on GitHub (Dec 20, 2016):
I ended up just uninstalling this package, but I have since run into this problem with some other packages.
For the other packages, I ended up working around the issue using the
core.customFileTypessetting inconfig.csonUnfortunately,
meteor-apidoesn't set a custom scope (source.js.meteor, for example), and instead uses the standardsource.js. Because of this I'm not sure if you'll be able to override it.This would work if the scope were set correctly, but you can try it anyway:
@hayk94 commented on GitHub (Jan 27, 2017):
Any fixes or workarounds yet?
@jacobmischka commented on GitHub (Jan 27, 2017):
You can fork this package, fix the scope like I mentioned then use the workaround I mentioned, and then install it with am via git.