Skip to content

test(oxlint): add --fix test case#6747

Merged
graphite-app[bot] merged 1 commit intomainfrom
10-21-test_oxlint_test_fix_test_case
Oct 21, 2024
Merged

test(oxlint): add --fix test case#6747
graphite-app[bot] merged 1 commit intomainfrom
10-21-test_oxlint_test_fix_test_case

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Oct 21, 2024

closes #6061

@graphite-app
Copy link
Contributor

graphite-app bot commented Oct 21, 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 added the 0-merge Merge with Graphite Merge Queue label Oct 21, 2024
@github-actions github-actions bot added A-cli Area - CLI C-test Category - Testing. Code is missing test cases, or a PR is adding them labels Oct 21, 2024
Copy link
Member Author

Boshen commented Oct 21, 2024

Merge activity

  • Oct 21, 11:23 AM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Oct 21, 11:37 AM EDT: A user added this pull request to the Graphite merge queue.
  • Oct 21, 11:42 AM EDT: A user merged this pull request with the Graphite merge queue.

@Boshen Boshen force-pushed the 10-21-test_oxlint_test_fix_test_case branch from ea38843 to efc94d7 Compare October 21, 2024 15:33
@Boshen Boshen force-pushed the 10-21-test_oxlint_test_fix_test_case branch from efc94d7 to b03cec6 Compare October 21, 2024 15:38
@graphite-app graphite-app bot merged commit b03cec6 into main Oct 21, 2024
@graphite-app graphite-app bot deleted the 10-21-test_oxlint_test_fix_test_case branch October 21, 2024 15:42
Boshen added a commit that referenced this pull request Oct 22, 2024
## [0.10.2] - 2024-10-22

### Features

- dbe1972 linter: Import/no-cycle should turn on ignore_types by default
(#6761) (Boshen)
- 619d06f linter: Fix suggestion for `eslint:no_empty_static_block` rule
(#6732) (Tapan Prakash)

### Bug Fixes


### Performance

- 8387bac linter: Apply small file optimization, up to 30% faster
(#6600) (camchenry)

### Refactor

- b884577 linter: All ast_util functions take Semantic (#6753)
(DonIsaac)
- 744aa74 linter: Impl `Deref<Target = Semantic>` for `LintContext`
(#6752) (DonIsaac)
- 6ffdcc0 oxlint: Lint/mod.rs -> lint.rs (#6746) (Boshen)

### Testing

- b03cec6 oxlint: Add `--fix` test case (#6747) (Boshen)

---------

Co-authored-by: Boshen <1430279+Boshen@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
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-cli Area - CLI 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.

Coworkers report that lint --fix will sometimes delete / empty / truncate a file.

1 participant