Skip to content

Conversation

@wChenonly
Copy link
Contributor

Summary

Related Links

fix: #322

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@Timeless0911
Copy link
Contributor

Can you write an unit test in

describe('transformSyntaxToRspackTarget', () => {

@Timeless0911 Timeless0911 requested a review from fi3ework October 23, 2024 09:42
@Timeless0911 Timeless0911 changed the title fix: map "es6" to "es2015" fix: map syntax "es6" to "es2015" Oct 23, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Oct 23, 2024

CodSpeed Performance Report

Merging #323 will not alter performance

Comparing wChenonly:main (09ef5a1) with main (6d74e88)

Summary

✅ 5 untouched benchmarks

@wChenonly wChenonly changed the title fix: map syntax "es6" to "es2015" fix: map syntax "es6" to "es2015" add test Oct 23, 2024
@wChenonly wChenonly changed the title fix: map syntax "es6" to "es2015" add test fix: map syntax "es6" to "es2015" Oct 23, 2024
@wChenonly
Copy link
Contributor Author

Can you write an unit test in

describe('transformSyntaxToRspackTarget', () => {

Copy link
Contributor

@Timeless0911 Timeless0911 left a comment

Choose a reason for hiding this comment

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

Thanks

@Timeless0911 Timeless0911 merged commit 8d5f047 into web-infra-dev:main Oct 23, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: set the syntax es6, rspack not match the API schema

2 participants