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

update target to ensure compilation does not alter generated code #34

Merged
merged 3 commits into from Jun 17, 2019

Conversation

rochdev
Copy link
Member

@rochdev rochdev commented Jun 14, 2019

This PR updates the TypeScript compilation target to the latest possible ECMAScript standard to ensure that the JavaScript output is not altered by the compiler. This will ensure that the generated JavaScript code is the same as the TypeScript code minus types.

The reasoning behind this has been discussed previously, but the short version is to avoid being unable to test certain features because they have been transpiled to something else.

Copy link
Member

@mayurkale22 mayurkale22 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the PR.

@mayurkale22 mayurkale22 merged commit e11d745 into open-telemetry:master Jun 17, 2019
pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this pull request Dec 15, 2023
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.

None yet

3 participants