Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

docs(linter): add docs for config settings #4827

Merged
merged 1 commit into from
Nov 21, 2024
Merged

docs(linter): add docs for config settings #4827

merged 1 commit into from
Nov 21, 2024
+195 −7

Conversation

DonIsaac
Copy link
Collaborator

No description provided.

@graphite-app Graphite App
Copy link

graphite-app bot commented Aug 11, 2024

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

Add the label “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.

@DonIsaac DonIsaac marked this pull request as ready for review August 11, 2024 16:01
@github-actions github-actions bot added the A-linter Area - Linter label Aug 11, 2024
@DonIsaac Graphite App
Copy link
Collaborator Author

DonIsaac commented Aug 11, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@DonIsaac DonIsaac mentioned this pull request Aug 11, 2024
Merged
@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Aug 11, 2024 •
edited
Loading

CodSpeed Performance Report

Merging #4827 will not alter performance

Comparing don/08-11-docs_linter_add_docs_for_config_settings (df143ca) with main (849489e)

Summary

✅ 30 untouched benchmarks

@DonIsaac DonIsaac force-pushed the don/08-11-feat_vscode_provide_config_s_schema_to_oxlint_config_files branch from 8506d03 to 5c08575 Compare August 11, 2024 16:18
@DonIsaac DonIsaac force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch from 93485a2 to bc152ca Compare August 11, 2024 16:18
@DonIsaac DonIsaac added the C-docs Category - Documentation. Related to user-facing or internal documentation label Aug 11, 2024
@DonIsaac DonIsaac force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch 2 times, most recently from 3deb42b to 5347420 Compare August 11, 2024 16:20
@DonIsaac DonIsaac force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch from 5347420 to eb98df2 Compare August 11, 2024 16:34
@DonIsaac
Copy link
Collaborator Author

Blocked by GREsau/schemars#313

@DonIsaac DonIsaac force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch from eb98df2 to 57b194b Compare August 11, 2024 16:53
@DonIsaac DonIsaac force-pushed the don/08-11-feat_vscode_provide_config_s_schema_to_oxlint_config_files branch from 5c08575 to 9a2736b Compare August 11, 2024 17:18
@DonIsaac DonIsaac force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch from 57b194b to c05c6b5 Compare August 11, 2024 17:18
@DonIsaac DonIsaac marked this pull request as draft August 11, 2024 17:33
Base automatically changed from don/08-11-feat_vscode_provide_config_s_schema_to_oxlint_config_files to main August 12, 2024 01:15
@Boshen Boshen closed this Nov 21, 2024
@Boshen Boshen reopened this Nov 21, 2024
@Boshen Boshen force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch from c05c6b5 to 2846c67 Compare November 21, 2024 07:44
@Boshen Boshen marked this pull request as ready for review November 21, 2024 07:44
@Boshen Boshen added the 0-merge Merge with Graphite Merge Queue label Nov 21, 2024
@Boshen Graphite App
Copy link
Member

Boshen commented Nov 21, 2024 •
edited by graphite-app bot
Loading

Merge activity

  • Nov 21, 2:44 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 21, 3:06 AM EST: A user added this pull request to the Graphite merge queue.
  • Nov 21, 3:13 AM EST: A user merged this pull request with the Graphite merge queue.

@Boshen Boshen force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch from 2846c67 to 54140d5 Compare November 21, 2024 08:01
@Boshen Boshen force-pushed the don/08-11-docs_linter_add_docs_for_config_settings branch from 54140d5 to df143ca Compare November 21, 2024 08:08
@graphite-app graphite-app bot merged commit df143ca into main Nov 21, 2024
26 checks passed
@graphite-app graphite-app bot deleted the don/08-11-docs_linter_add_docs_for_config_settings branch November 21, 2024 08:13
@oxc-bot oxc-bot mentioned this pull request Nov 21, 2024
Merged
Boshen added a commit that referenced this pull request Nov 21, 2024
@oxc-bot @Boshen
release(oxlint): v0.13.0 (#7398)
4b5a176 
## [0.13.0] - 2024-11-21

- 7bf970a linter: [**BREAKING**] Remove tree_shaking plugin (#7372)
(Boshen)
- 7f8747d linter: Implement `react/no-array-index-key` (#6960)
(BitterGourd)

### Features

- be152c0 linter: Add `typescript/no-require-imports` rule (#7315)
(Dmitry Zakharov)
- 849489e linter: Add suggestion for no-console (#4312) (DonIsaac)
- 8cebdc8 linter: Allow appending plugins in override (#7379)
(camchenry)
- 8cfea3c oxc_cfg: Add implicit return instruction (#5568)
(IWANABETHATGUY)
- e6922df parser: Fix incorrect AST for `x?.f<T>()` (#7387) (Boshen)

### Bug Fixes

- e91c287 linter: Fix panic in react/no-array-index-key (#7395) (Boshen)
- a32f5a7 linter/no-array-index-key: Compile error due to it uses a
renamed API (#7391) (Dunqing)
- 666b6c1 parser: Add missing `ChainExpression` in optional
`TSInstantiationExpression` (#7371) (Boshen)

### Documentation

- df143ca linter: Add docs for config settings (#4827) (DonIsaac)
- ad44cfa linter: Import/first options (#7381) (Zak)

### Refactor

- c34d649 linter: Use `scope_id` etc methods (#7394) (overlookmotel)
- 466f395 vscode: Split `ConfigService` and `Config` (#7376) (Alexander
S.)

Co-authored-by: Boshen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
0-merge Merge with Graphite Merge Queue A-linter Area - Linter C-docs Category - Documentation. Related to user-facing or internal documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DonIsaac @Boshen