-
Notifications
You must be signed in to change notification settings - Fork 131
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
Eslint/Rollup #110
base: master
Are you sure you want to change the base?
Eslint/Rollup #110
Conversation
@michaelficarra : If you are able to review, the ones here on estraverse would probably be helpful to have done first so I can simplify the Rollup routine on its dependees (to avoid need for the CJS plugin) Update: Probably should resolve concerns at estools/esutils#34 first actually |
HI. Can you please duplicate this PR to my fork? |
- npm: Add recommended `bugs`, `keywords`, `dependencies` and use `author`/`contributors` - npm: Update devDeps, and switch to Babel 7/preset-env - npm: Use simple run script syntax - npm: Drop `package-lock.json` in favor of npmrc for consistency with other estools projects - Maintenance: Add `.editorconfig` - Maintenance: Drop use of gulpfile - Update: Mocha API - Testing: Add nyc - Testing: Use `chai/register-expect` - Testing: Drop bundled `esprima` in favor of using versioned - Testing: Update espree API to supply `sourceType`/`ecmaVersion` - Testing: Enable `espree` for dynamic import and adjust test expectation to take into account full Program context - Docs: Use more modern syntax in examples
…t`); prefer for-of - Refactoring: Use ES6 classes
- Refactoring: Switch to ESM source, using Rollup + Babel/Terser - Refactoring: Further ES6 class
I've added native Node ESM support. @sanex3339 : I'm planning on maintaining my own fork (already started at |
Builds on #109.
exports
exports
let
closer to scope of use (or convert toconst
); prefer for-of