Skip to content

Conversation

@TheAlexLichter
Copy link
Contributor

@TheAlexLichter TheAlexLichter commented Sep 18, 2025

Adds .oxlintrc.json, Oxlints config file to JSON with comments entry.

Checklist:

@TheAlexLichter TheAlexLichter requested a review from a team as a code owner September 18, 2025 20:03
@lildude lildude changed the title chore: add .oxlintrc.json to json with comments Add .oxlintrc.json to json with comments Oct 6, 2025
Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

You need to add the sample you reference to this PR.

@Alhadis Alhadis changed the title Add .oxlintrc.json to json with comments Add .oxlintrc.json to JSON with comments Oct 16, 2025
@ArnaudBarre
Copy link
Contributor

A real world usage (not from the docs) with a comment in a repo with more than 18k star: https://github.com/monkeytypegame/monkeytype/blob/2f5cb164cbb9ad2c6ab1242053f1f23fdb34f6f2/frontend/.oxlintrc.json
(found in the first page of results provided by @TheAlexLichter)

@lildude lildude enabled auto-merge December 12, 2025 11:33
@ArnaudBarre
Copy link
Contributor

@lildude I think the merge is blocked by the requested change review?

@lildude
Copy link
Member

lildude commented Dec 12, 2025

Oh yes. Whoops.

Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

Important

The changes in this PR will not appear on GitHub until the next release has been made and deployed. See here for more details.

@lildude lildude added this pull request to the merge queue Dec 12, 2025
Merged via the queue into github-linguist:main with commit a7e40d3 Dec 12, 2025
5 checks passed
@TheAlexLichter TheAlexLichter deleted the chore/add-oxlintrc branch December 12, 2025 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants