Skip to content

fix(ast): few serialization issues#2522

Merged
Boshen merged 1 commit intooxc-project:mainfrom
ArnaudBarre:more-camel-case
Feb 27, 2024
Merged

fix(ast): few serialization issues#2522
Boshen merged 1 commit intooxc-project:mainfrom
ArnaudBarre:more-camel-case

Conversation

@ArnaudBarre
Copy link
Copy Markdown
Contributor

@ArnaudBarre ArnaudBarre commented Feb 27, 2024

A few more I missed in #2506. Re #2463.

Only remaining snake_case in the current types of the AST: trailing_comma in ArrayExpression, ObjectExpression & ArrayAssignmentTarget. I waiting for the discussion on #2521 to choose between skip and camelCase.

@github-actions github-actions bot added the A-ast Area - AST label Feb 27, 2024
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Feb 27, 2024

CodSpeed Performance Report

Merging #2522 will not alter performance

Comparing ArnaudBarre:more-camel-case (2cda62c) with main (46e7791)

Summary

✅ 27 untouched benchmarks

@Boshen Boshen marked this pull request as ready for review February 27, 2024 05:03
@Boshen Boshen changed the title Fix few serialization issues fix(ast0: fix few serialization issues Feb 27, 2024
@Boshen Boshen changed the title fix(ast0: fix few serialization issues fix(ast): few serialization issues Feb 27, 2024
@Boshen Boshen merged commit 6d5ec6d into oxc-project:main Feb 27, 2024
@ArnaudBarre ArnaudBarre deleted the more-camel-case branch February 27, 2024 10:31
IWANABETHATGUY pushed a commit to IWANABETHATGUY/oxc that referenced this pull request May 29, 2024
A few more I missed in oxc-project#2506. Re oxc-project#2463.

Only remaining snake_case in the current types of the AST:
`trailing_comma` in ArrayExpression, ObjectExpression &
ArrayAssignmentTarget.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ast Area - AST

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants