Skip to content

Comments

fix(formatter): correct a bunch of implementations for TypeScript#14069

Merged
graphite-app[bot] merged 1 commit intomainfrom
09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript
Sep 24, 2025
Merged

fix(formatter): correct a bunch of implementations for TypeScript#14069
graphite-app[bot] merged 1 commit intomainfrom
09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript

Conversation

@Dunqing
Copy link
Member

@Dunqing Dunqing commented Sep 24, 2025

No description provided.

Copy link
Member Author

Dunqing commented Sep 24, 2025

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 24, 2025

CodSpeed Instrumentation Performance Report

Merging #14069 will not alter performance

Comparing 09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript (7d11047) with main (6cfce80)1

Summary

✅ 37 untouched

Footnotes

  1. No successful run was found on main (7d11047) during the generation of this report, so 6cfce80 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@Dunqing Dunqing marked this pull request as ready for review September 24, 2025 06:24
@Dunqing Dunqing force-pushed the 09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript branch from 4249712 to 2a1d453 Compare September 24, 2025 07:06
@Dunqing Dunqing force-pushed the 09-23-feat_formatter_implement_formatting_for_tstypealiasdeclaration branch from 1479d8e to 0898d4d Compare September 24, 2025 07:06
@graphite-app graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Sep 24, 2025
@graphite-app graphite-app bot force-pushed the 09-23-feat_formatter_implement_formatting_for_tstypealiasdeclaration branch 2 times, most recently from aa2617a to 0efacb3 Compare September 24, 2025 09:21
@graphite-app graphite-app bot changed the base branch from 09-23-feat_formatter_implement_formatting_for_tstypealiasdeclaration to graphite-base/14069 September 24, 2025 09:28
@leaysgur leaysgur force-pushed the 09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript branch from 2a1d453 to 6ea768d Compare September 24, 2025 09:28
@graphite-app graphite-app bot changed the base branch from graphite-base/14069 to main September 24, 2025 09:29
@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 24, 2025

Merge activity

  • Sep 24, 9:29 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.
  • Sep 24, 10:27 AM UTC: leaysgur added this pull request to the Graphite merge queue.
  • Sep 24, 10:33 AM UTC: Merged by the Graphite merge queue.

@graphite-app graphite-app bot force-pushed the 09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript branch from 6ea768d to c71ecb2 Compare September 24, 2025 09:34
Copilot AI review requested due to automatic review settings September 24, 2025 10:28
@graphite-app graphite-app bot force-pushed the 09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript branch from c71ecb2 to 7d11047 Compare September 24, 2025 10:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@graphite-app graphite-app bot merged commit 7d11047 into main Sep 24, 2025
25 checks passed
@graphite-app graphite-app bot deleted the 09-24-fix_formatter_correct_a_bunch_of_implementations_for_typescript branch September 24, 2025 10:33
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Sep 24, 2025
@Boshen Boshen mentioned this pull request Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ast-tools Area - AST tools A-formatter Area - Formatter C-bug Category - Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant