Skip to content

refactor(ast): shorten code for impl Serialize for Elision#8940

Merged
graphite-app[bot] merged 1 commit intomainfrom
02-06-refactor_ast_shorten_code_for_impl_serialize_for_elision_
Feb 7, 2025
Merged

refactor(ast): shorten code for impl Serialize for Elision#8940
graphite-app[bot] merged 1 commit intomainfrom
02-06-refactor_ast_shorten_code_for_impl_serialize_for_elision_

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Feb 6, 2025

Small refactor. Just shorten code.

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 6, 2025

CodSpeed Performance Report

Merging #8940 will not alter performance

Comparing 02-06-refactor_ast_shorten_code_for_impl_serialize_for_elision_ (515eb52) with 02-06-refactor_ast_introduce_estree_ts_alias_attr_and_use_it_on_elision_ (640db88)

Summary

✅ 33 untouched benchmarks

@Dunqing Dunqing added the 0-merge Merge with Graphite Merge Queue label Feb 7, 2025
Copy link
Member

Dunqing commented Feb 7, 2025

Merge activity

  • Feb 6, 9:46 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Feb 6, 9:46 PM EST: A user added this pull request to the Graphite merge queue.
  • Feb 6, 10:15 PM EST: A user merged this pull request with the Graphite merge queue.

@graphite-app graphite-app bot force-pushed the 02-06-refactor_ast_introduce_estree_ts_alias_attr_and_use_it_on_elision_ branch from b62cec2 to 640db88 Compare February 7, 2025 02:49
@graphite-app graphite-app bot force-pushed the 02-06-refactor_ast_shorten_code_for_impl_serialize_for_elision_ branch from 44cd703 to 515eb52 Compare February 7, 2025 02:50
Base automatically changed from 02-06-refactor_ast_introduce_estree_ts_alias_attr_and_use_it_on_elision_ to main February 7, 2025 03:12
@graphite-app graphite-app bot merged commit 515eb52 into main Feb 7, 2025
28 checks passed
@graphite-app graphite-app bot deleted the 02-06-refactor_ast_shorten_code_for_impl_serialize_for_elision_ branch February 7, 2025 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0-merge Merge with Graphite Merge Queue A-ast Area - AST C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants