Skip to content

docs: use recommended-typescript for typescript files #949

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.

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

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

bluwy
Copy link
Contributor

@bluwy bluwy commented Feb 26, 2025

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Docs
  • New Binding issue #___
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes, and the changes were approved in issue #___
  • No

Checklist

  • When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
  • I have added a convincing reason for adding this feature, if necessary

Other information

IIUC the recommended-typescript config is better for setups with typescript-eslint or ts/tsx files in general, so I updated the docs to use it. If recommended is intentional, I'm happy to close this PR. I figured it may have been overlooked.

I did not sign my commits as according the the contributing guide 😬 Shouldn't it be signed by github after squash-merging to main?

Sorry, something went wrong.

Copy link

vercel bot commented Feb 26, 2025

@bluwy is attempting to deploy a commit to the Rel1cx's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Feb 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eslint-react ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 26, 2025 3:23pm

@Rel1cx Rel1cx added the Type: Documentation Improvements or additions to documentation label Feb 26, 2025
@Rel1cx
Copy link
Owner

Rel1cx commented Feb 26, 2025

LGTM. signed by github after squash-merging is also acceptable.

@Rel1cx Rel1cx merged commit 0beaadc into Rel1cx:main Feb 26, 2025
8 of 9 checks passed
@bluwy bluwy deleted the recommend-typescript branch February 26, 2025 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants