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

add support for optional chaining #412

Merged

Conversation

sanex3339
Copy link
Contributor

@sanex3339 sanex3339 commented May 30, 2020

@sanex3339
Copy link
Contributor Author

acornjs/acorn#891 merged, will back to this PR this weekend

@sanex3339
Copy link
Contributor Author

Updated this PR

@sanex3339
Copy link
Contributor Author

@michaelficarra 👀

@sanex3339 sanex3339 changed the title Optional chaining support WIP Optional chaining support Jun 11, 2020
@sanex3339 sanex3339 closed this Jun 12, 2020
@sanex3339 sanex3339 reopened this Jun 12, 2020
@sanex3339
Copy link
Contributor Author

@michaelficarra should we support node@4 in travis? I added chai-exclude package to deep compare AST-trees without start and end properties, but looks like it doesn't support node@4.

I can remove node@4 from travis.yml and add node@10 instead

@sanex3339
Copy link
Contributor Author

sanex3339 commented Jun 12, 2020

Ok. I removed node@4 from travis.yml file and added node@10 instead.
Also, I updated engines section with >=6.0 value.

Node 4 is few years as deprecated, so I think it's the time to leave this version

@sanex3339
Copy link
Contributor Author

@michaelficarra also i created PR to the estraverse to support ChainExpression node.
estools/estraverse#113

So it is best to merge estravserse PR first and after that, I'll update this PR with the use of Syntax.ChainExpression.

@sanex3339
Copy link
Contributor Author

Any news?

@sanex3339
Copy link
Contributor Author

Any ETA when this PR can be reviewed?

@dilame
Copy link

dilame commented Jun 23, 2020

Hi guys! This PR is very important for me and many people in the world! Please, merge it

@mikhailrojo
Copy link

I upvote for fast merge

@sanex3339
Copy link
Contributor Author

sanex3339 commented Jun 24, 2020

@Constellation Hi. Can you check and merge this?

@sanex3339
Copy link
Contributor Author

@michaelficarra re-ping

1 similar comment
@sanex3339
Copy link
Contributor Author

@michaelficarra re-ping

@dilame
Copy link

dilame commented Jun 29, 2020

i support this re-ping

@sanex3339
Copy link
Contributor Author

sanex3339 commented Jul 2, 2020

@michaelficarra please, re-check this. It is REALLY hard to contribute to this project because of a lack of maintenance. It kills all motivation to contribute here.

@sanex3339
Copy link
Contributor Author

@michaelficarra re-ping

1 similar comment
@dilame
Copy link

dilame commented Jul 6, 2020

@michaelficarra re-ping

@sanex3339
Copy link
Contributor Author

@michaelficarra ping

@sanex3339
Copy link
Contributor Author

image
Still waiting...

@sanex3339
Copy link
Contributor Author

@michaelficarra ping

@sanex3339
Copy link
Contributor Author

@michaelficarra please check this. More than a month I'm waiting for your approval.

@dilame
Copy link

dilame commented Jul 20, 2020

@michaelficarra it is insulting the feelings of contributors!

@sanex3339 sanex3339 force-pushed the optional-chaining-support branch from 4cce5f8 to f672353 Compare August 5, 2020 19:01
@sanex3339
Copy link
Contributor Author

@sanex3339 Please rebase.

@michaelficarra done

@michaelficarra michaelficarra changed the title Optional chaining support add support for optional chaining Aug 5, 2020
@michaelficarra michaelficarra merged commit 1d192ae into estools:master Aug 5, 2020
@michaelficarra
Copy link
Member

Published v2.0.0 (since I got yelled at last time I dropped support for an ancient node version in a minor release).

@sanex3339
Copy link
Contributor Author

Thank you!

sanex3339 added a commit to javascript-obfuscator/escodegen that referenced this pull request Aug 14, 2020
Co-authored-by: sanex3339 <yarabotayuvyandex3339>
brettz9 added a commit to brettz9/escodegen that referenced this pull request Feb 9, 2021
Merge branch 'eslint-npm' into eslint-npm-full

* eslint-npm:
  - npm: update estraverse - npm: update devDeps. - npm: Update package-lock version
  - npm: update devDeps. - npm: Update package-lock version
  - Linting: Temporarily disable most linting, with a few fixes
  - Breaking change: Remove bower-registry-client build (bower deprecated) - Update: Use new SourceMapConsumer API in test - Build: Provide browserified builds with npm package - Travis: Drop 4, 6, 8; Add 10, 12, 14; check build - Maintenance: Add `.editorconfig` - Docs: Use fenced code blocks in README (for syntax highlighting) - npm: Add `bugs`, `keywords`, change from `maintainers` to `authors`/`contributors` - npm: Restore `optionator` to a regular dep. (used in published binary file) - npm: Drop unused semver, minimist - npm: Bump deps. (estraverse, optionator, optional source-map potentially breaking) and devDeps. - npm: Drop bluebird in favor of ES Promises - npm: Use more recently maintained browserify + uglifyify - npm: Replace linting and testing scripts in Gulpfile with npm scripts
  Add more optional-chaining tests
  Test logical assignments
  Implement coalescing code generation
  Support BigInt syntax
  Update gulpfile and dependencies
  Version 2.0.0
  add support for optional chaining (estools#412)
  Update .gitattributes
  drop support for node <6 (estools#419)

# Conflicts:
#	.eslintignore
#	.eslintrc.js
#	.gitignore
#	escodegen.js.map
#	package-lock.json
#	package.json
#	src/escodegen.js
#	test/source-map.js
#	tools/release.js
brettz9 added a commit to brettz9/escodegen that referenced this pull request Feb 9, 2021
Merge branch 'eslint-npm' into eslint-npm-full; also fixes regression (line 959)
introduced in cf9a7b8

* eslint-npm:
  - npm: update estraverse - npm: update devDeps. - npm: Update package-lock version
  - npm: update devDeps. - npm: Update package-lock version
  - Linting: Temporarily disable most linting, with a few fixes
  - Breaking change: Remove bower-registry-client build (bower deprecated) - Update: Use new SourceMapConsumer API in test - Build: Provide browserified builds with npm package - Travis: Drop 4, 6, 8; Add 10, 12, 14; check build - Maintenance: Add `.editorconfig` - Docs: Use fenced code blocks in README (for syntax highlighting) - npm: Add `bugs`, `keywords`, change from `maintainers` to `authors`/`contributors` - npm: Restore `optionator` to a regular dep. (used in published binary file) - npm: Drop unused semver, minimist - npm: Bump deps. (estraverse, optionator, optional source-map potentially breaking) and devDeps. - npm: Drop bluebird in favor of ES Promises - npm: Use more recently maintained browserify + uglifyify - npm: Replace linting and testing scripts in Gulpfile with npm scripts
  Add more optional-chaining tests
  Test logical assignments
  Implement coalescing code generation
  Support BigInt syntax
  Update gulpfile and dependencies
  Version 2.0.0
  add support for optional chaining (estools#412)
  Update .gitattributes
  drop support for node <6 (estools#419)

# Conflicts:
#	.eslintignore
#	.eslintrc.js
#	.gitignore
#	escodegen.js.map
#	package-lock.json
#	package.json
#	src/escodegen.js
#	test/source-map.js
#	tools/release.js
brettz9 added a commit to brettz9/escodegen that referenced this pull request Feb 9, 2021
Merge branch 'eslint-npm' into eslint-npm-full; also adds linting to recent adds and
fixes regression (line 959) introduced in
cf9a7b8

* eslint-npm:
  - npm: update estraverse - npm: update devDeps. - npm: Update package-lock version
  - npm: update devDeps. - npm: Update package-lock version
  - Linting: Temporarily disable most linting, with a few fixes
  - Breaking change: Remove bower-registry-client build (bower deprecated) - Update: Use new SourceMapConsumer API in test - Build: Provide browserified builds with npm package - Travis: Drop 4, 6, 8; Add 10, 12, 14; check build - Maintenance: Add `.editorconfig` - Docs: Use fenced code blocks in README (for syntax highlighting) - npm: Add `bugs`, `keywords`, change from `maintainers` to `authors`/`contributors` - npm: Restore `optionator` to a regular dep. (used in published binary file) - npm: Drop unused semver, minimist - npm: Bump deps. (estraverse, optionator, optional source-map potentially breaking) and devDeps. - npm: Drop bluebird in favor of ES Promises - npm: Use more recently maintained browserify + uglifyify - npm: Replace linting and testing scripts in Gulpfile with npm scripts
  Add more optional-chaining tests
  Test logical assignments
  Implement coalescing code generation
  Support BigInt syntax
  Update gulpfile and dependencies
  Version 2.0.0
  add support for optional chaining (estools#412)
  Update .gitattributes
  drop support for node <6 (estools#419)

# Conflicts:
#	.eslintignore
#	.eslintrc.js
#	.gitignore
#	escodegen.js.map
#	package-lock.json
#	package.json
#	src/escodegen.js
#	test/source-map.js
#	tools/release.js
brettz9 added a commit to brettz9/escodegen that referenced this pull request Feb 9, 2021
…recent adds and

fixes regression (line 959) introduced in
cf9a7b8

* eslint-npm:
  - npm: update estraverse - npm: update devDeps. - npm: Update package-lock version
  - npm: update devDeps. - npm: Update package-lock version
  - Linting: Temporarily disable most linting, with a few fixes
  - Breaking change: Remove bower-registry-client build (bower deprecated) - Update: Use new SourceMapConsumer API in test - Build: Provide browserified builds with npm package - Travis: Drop 4, 6, 8; Add 10, 12, 14; check build - Maintenance: Add `.editorconfig` - Docs: Use fenced code blocks in README (for syntax highlighting) - npm: Add `bugs`, `keywords`, change from `maintainers` to `authors`/`contributors` - npm: Restore `optionator` to a regular dep. (used in published binary file) - npm: Drop unused semver, minimist - npm: Bump deps. (estraverse, optionator, optional source-map potentially breaking) and devDeps. - npm: Drop bluebird in favor of ES Promises - npm: Use more recently maintained browserify + uglifyify - npm: Replace linting and testing scripts in Gulpfile with npm scripts
  Add more optional-chaining tests
  Test logical assignments
  Implement coalescing code generation
  Support BigInt syntax
  Update gulpfile and dependencies
  Version 2.0.0
  add support for optional chaining (estools#412)
  Update .gitattributes
  drop support for node <6 (estools#419)

# Conflicts:
#	.eslintignore
#	.eslintrc.js
#	.gitignore
#	escodegen.js.map
#	package-lock.json
#	package.json
#	src/escodegen.js
#	test/source-map.js
#	tools/release.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants