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

Dynamic wildcard analysis #40

Open
guybedford opened this issue May 23, 2019 · 2 comments
Open

Dynamic wildcard analysis #40

guybedford opened this issue May 23, 2019 · 2 comments
Labels
enhancement New feature or request priority Important issue or pull request to fast-track

Comments

@guybedford
Copy link
Contributor

With the PR in #39 we can now support wildcard requires like require('./local' + name), but for cases like:

const requirePath = './local' + name;
require(requirePath)

the wildcard analysis won't extend to the variable tracing.

We can add support for this by extending the variable analysis to not be known static values, but also partial values as well, although this does involve refactoring the value tracking system somewhat.

@guybedford guybedford added the enhancement New feature or request label May 23, 2019
@styfle styfle added the priority Important issue or pull request to fast-track label May 24, 2019
@demiban
Copy link

demiban commented Apr 28, 2020

Any update on this issue?

@quanru
Copy link

quanru commented Oct 27, 2023

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority Important issue or pull request to fast-track
Projects
None yet
Development

No branches or pull requests

4 participants