Skip to content

Comments

test(semantic): re-organize snapshot-based tests#7494

Merged
graphite-app[bot] merged 1 commit intomainfrom
11-26-test_semantic_re-organize_snapshot-based_tests
Nov 27, 2024
Merged

test(semantic): re-organize snapshot-based tests#7494
graphite-app[bot] merged 1 commit intomainfrom
11-26-test_semantic_re-organize_snapshot-based_tests

Conversation

@Dunqing
Copy link
Member

@Dunqing Dunqing commented Nov 26, 2024

Reorignize tests by splitting tests to js, ts, and jsx folders

@Dunqing Dunqing requested a review from Boshen as a code owner November 26, 2024 15:33
@github-actions github-actions bot added A-semantic Area - Semantic C-test Category - Testing. Code is missing test cases, or a PR is adding them labels Nov 26, 2024
Copy link
Member Author

Dunqing commented Nov 26, 2024

@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 26, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@Boshen Boshen changed the base branch from 10-09-fix_semantic_correctly_resolve_binding_for_return_type_of_functions to graphite-base/7494 November 26, 2024 15:43
@Boshen Boshen force-pushed the 11-26-test_semantic_re-organize_snapshot-based_tests branch from 120a2fc to b4786d9 Compare November 26, 2024 15:54
@Boshen Boshen force-pushed the graphite-base/7494 branch from dd514ba to f3850eb Compare November 26, 2024 15:54
@Boshen Boshen changed the base branch from graphite-base/7494 to main November 26, 2024 15:55
@Boshen Boshen force-pushed the 11-26-test_semantic_re-organize_snapshot-based_tests branch from b4786d9 to 7580e45 Compare November 26, 2024 15:55
Copy link
Member

@overlookmotel overlookmotel left a comment

Choose a reason for hiding this comment

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

I don't know how the semantic fixtures work, so can't review. But it looks reasonable. @Dunqing if you're happy with it, please merge.

@graphite-app graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Nov 27, 2024
@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 27, 2024

Merge activity

Reorignize tests by splitting tests to js, ts, and jsx folders
@Dunqing Dunqing force-pushed the 11-26-test_semantic_re-organize_snapshot-based_tests branch from 7580e45 to 4b0720a Compare November 27, 2024 02:01
@graphite-app graphite-app bot merged commit 4b0720a into main Nov 27, 2024
@graphite-app graphite-app bot deleted the 11-26-test_semantic_re-organize_snapshot-based_tests branch November 27, 2024 02:05
@oxc-bot oxc-bot mentioned this pull request Dec 4, 2024
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-semantic Area - Semantic C-test Category - Testing. Code is missing test cases, or a PR is adding them

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants