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

[FEATURE] jsdoc: Improve support for ES6+ syntax #785

Merged
merged 27 commits into from
Sep 9, 2022

Conversation

d3xter666
Copy link
Contributor

JIRA: CPOUI5FOUNDATION-374

There's no such thing as Syntax.OptionalMemberExpression/OptionalCallExpression, but JSDoc parser provides such "types" in case of testing.
So, we need to stringify those in order to catch those cases
Some expressions might be wrappers, like ChainExpression, AwaitExpression.
If we do not need to make something very specific with them, we need to resolve the underlying structures
@d3xter666 d3xter666 changed the title [WIP][FEATURE] ES6+ support jsdoc plugin [WIP][INTERNAL] ES6+ support jsdoc plugin Aug 22, 2022
@d3xter666 d3xter666 changed the title [WIP][INTERNAL] ES6+ support jsdoc plugin [WIP][INTERNAL] Enhance ES6+ support jsdoc plugin Aug 22, 2022
@coveralls
Copy link

coveralls commented Aug 22, 2022

Coverage Status

Coverage remained the same at 94.042% when pulling 27bfd49 on next-es6-support-jsdoc-plugin into a6146a5 on next.

@d3xter666 d3xter666 changed the title [WIP][INTERNAL] Enhance ES6+ support jsdoc plugin [INTERNAL] Enhance ES6+ support jsdoc plugin Aug 24, 2022
- only withput expression
- treated like normal strings
@d3xter666 d3xter666 changed the title [INTERNAL] Enhance ES6+ support jsdoc plugin [WIP][INTERNAL] Enhance ES6+ support jsdoc plugin Aug 24, 2022
@d3xter666 d3xter666 changed the title [WIP][INTERNAL] Enhance ES6+ support jsdoc plugin [INTERNAL] Enhance ES6+ support jsdoc plugin Aug 30, 2022
@d3xter666 d3xter666 requested a review from matz3 September 7, 2022 12:58
@matz3 matz3 changed the title [INTERNAL] Enhance ES6+ support jsdoc plugin [FEATURE] jsdoc: Improve support for ES6+ Syntax Sep 8, 2022
@matz3 matz3 changed the title [FEATURE] jsdoc: Improve support for ES6+ Syntax [FEATURE] jsdoc: Improve support for ES6+ syntax Sep 8, 2022
Copy link
Member

@matz3 matz3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use squash and merge and ensure that the message title uses [FEATURE] (as in the PR title)

@d3xter666 d3xter666 merged commit 187a6a3 into next Sep 9, 2022
@d3xter666 d3xter666 deleted the next-es6-support-jsdoc-plugin branch September 9, 2022 05:45
openui5bot pushed a commit to SAP/openui5 that referenced this pull request Oct 5, 2022
This change integrates the following Pull request from ui5-builder
project: SAP/ui5-builder#785

Change-Id: I3174df8c47a95f52593d544787ee73079f99ca98
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.

3 participants