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

refactor: update documentation about generating JS examples #1348

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

danielpeintner
Copy link
Member

Update the JS output accordingly also

fixes #1347

@danielpeintner
Copy link
Member Author

Mhh, the failing "coding style check" seems to have something to do with my local setup. See also #1349

Copy link
Member

@relu91 relu91 left a comment

Choose a reason for hiding this comment

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

I had similar problems in previous PRs, it might be a misalignment with the version of prettier used by CI and the one used by our local stack.

About the PR itself, I found some issues below

@@ -13,6 +13,7 @@
*
* SPDX-License-Identifier: EPL-2.0 OR W3C-20150513
********************************************************************************/
Object.defineProperty(exports, "__esModule", { value: true });
Copy link
Member

Choose a reason for hiding this comment

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

This should not be there (according to the tests that I did in the past), I'll double-check locally to understand why it generates this line now.

/********************************************************************************
* Copyright (c) 2022 Contributors to the Eclipse Foundation
* Copyright (c) 2021 Contributors to the Eclipse Foundation
Copy link
Member

Choose a reason for hiding this comment

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

These license changes are ok? should we maybe update the ones coming from TS ? are they good?

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.

Outdated documentation about generating examples
2 participants