fisker Cheung

Results 1275 comments of fisker Cheung

I think these cases are both `ExpressionStatement` in AST, how about a rule named `no-restricted-statements`, forbid `arr.map(item => item.name)` as a `ExpressionStatement`, and a configurable `denyList` accepts [esquery](https://github.com/estools/esquery) selector ```js...

> We already has "globalReturn" option, we could just add "globalUsing" option as well, without touching sourceType? That should work, but the ecosystem prefers one flag for both, ESPree introduced...

> should we add parseCommonJs? I prefer to remove `parseScript` and `parseModule`, only keep `parse` > Do we need to do something about "globalReturn" option Deprecate, and remove later?

Acorn also deprecating `allowReturnOutsideFunction` https://github.com/acornjs/acorn/issues/1376#issuecomment-2960087991

We have already made a decision, let's start? Should I just commit to main branch?

I'm afraid I don't have enough knowledge to add support for `using`, hope you or someone else can add it in v7.

@seiyab Do you know we have `doc-explorer` parser in playground? 😄

* [x] https://github.com/tc39/test262/blob/823f4cfd2004cf410bc0344304e22e12d243e955/test/language/import/import-attributes/json-named-bindings.js https://github.com/meriyah/meriyah/pull/484 * [ ] https://github.com/tc39/test262/blob/823f4cfd2004cf410bc0344304e22e12d243e955/test/language/statements/await-using/throws-if-initializer-not-object.js `await using` not supported yet * [ ] https://github.com/tc39/test262/blob/823f4cfd2004cf410bc0344304e22e12d243e955/test/language/statements/class/elements/syntax/valid/grammar-field-named-get-followed-by-generator-asi.js https://github.com/meriyah/meriyah/issues/485 * [ ] https://github.com/tc39/test262/blob/823f4cfd2004cf410bc0344304e22e12d243e955/test/language/statements/for-await-of/head-lhs-async.js in whitelist * [ ] https://github.com/tc39/test262/blob/823f4cfd2004cf410bc0344304e22e12d243e955/test/language/statements/using/throws-if-initializer-not-object.js `using` not...

I just checked the history, it's added in https://github.com/prettier/prettier/pull/2733 with lots of questions, basically, it was added only because someone may use the CLI without a config file. I highly...

I agree to remove it. CLI flags should always take higher precedence.