-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
expose semi-core modules via resolvers #479
Comments
Any progress on this? |
@merlinpatt No, no progress. If you feel like helping out, there could be some :) |
@jfmengels do you have suggestions on where to look to get started? I have 2 years dev experience but I've never written or contributed to a resolver before |
Thanks for considering contributing! Unfortunately, that's a part I haven't touched either, and I don't want to send you on the wrong path. @benmosher or @tleunen could probably advise you. |
@merlinpatt I appreciate your enthusiasm! I am not sure this is a good place for a first contribution, though, as it's mostly an API change. This is top of mind for me now that v2 is released, though, so I will be on it ASAP. 😁 |
Any progress to solve this issue? |
clayne11/eslint-import-resolver-meteor#11
TL;DR: semi-magic modules (in this case,
meteor/*
) that exist on disk (and thus can be linted) but are baked into the environment (and thus, are not explicitly dependencies) ideally should not blow upno-extraneous-dependencies
.Hand-wavy possible solution: allow resolvers to define import types somehow. (maybe move the
importType
functionality down into the resolver spec?)Or could keep the existing behavior when/if chosen resolver does not explicitly provide the designation.
The text was updated successfully, but these errors were encountered: