Doesn't handle var statements with multiple declarations
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 45/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- javascript
- Domain
- tooling
Research direction
Start by reproducing the failure with the multi-declaration var example and compare it with the separate declarations example. Trace static-eval's handling of var statements; the issue is done when multiple declarations no longer throw while the existing single-declaration case continues to work.
Written by the indexing model from the issue text.
Description
I can't bulkify code that looks like this because static-eval is throwing:
var bulk = require('bulk-require')
, _ = require('lodash')
, conditions = bulk(__dirname, ['/*.js'])
However, this works:
var bulk = require('bulk-require')
var _ = require('lodash')
var conditions = bulk(__dirname, ['/*.js'])
- Dominant language
- JavaScript
- Stars
- 177
- Forks
- 27
- PR merge metrics
- No merged PRs in 30d
Getting set up
We have not checked this project's setup files yet. Start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from browserify/static-eval
-
CVE in word-wrapOpen
Difficulty 3/5 1-2 days Newbie friendliness 35/100
browserify/static-eval#42 · 1 comment · 2 reactions ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
browserify/static-eval#41 · 1 comment ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
browserify/static-eval#39 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
browserify/static-eval#38 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
browserify/static-eval#34 · 20 comments · 3 reactions ·
All issues in browserify/static-eval
Similar issues
-
[Bug] @deck.gl/arcgis dist import resolves to unpublished @deck.gl/core source path (9.3.11, 9.4.0)Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
stac-utils/stac-fields#42 ·
-
[sec-check] SVG remote-reference gate misses src attributes, link hrefs and bare-string @importOpenagent/security hive/hosted-available-lke648397-260827-5n31 security
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
Maintainers usually reply within 1 day
-
channels:add check:passed
Difficulty 1/5 Under an hour Newbie friendliness 75/100
Maintainers usually reply within 2 days