Skip to content

fix: lint and format generated files using stricter config#2181

Open
charlesfries wants to merge 1 commit intot3-oss:mainfrom
charlesfries:format
Open

fix: lint and format generated files using stricter config#2181
charlesfries wants to merge 1 commit intot3-oss:mainfrom
charlesfries:format

Conversation

@charlesfries
Copy link
Copy Markdown

✅ Checklist

  • I have followed every step in the contributing guide (updated 2022-10-06).
  • The PR title follows the convention we established conventional-commit
  • I performed a functional test on my final commit

Changelog

Switching to typescript-eslint strictTypeChecked config in a generated app reveals a handful of simple lint errors. This PR fixes those issues. Additionally, the generated Prettier write script omits CSS files, and the generated CSS is missing a single newline, which is also fixed in this PR.

  • Remove unnecessary template literal usage (@typescript-eslint/no-unnecessary-template-expression)
  • Cast numbers to strings in console.log (@typescript-eslint/restrict-template-expressions)
  • Format globals.css using Prettier

…@typescript-eslint/restrict-template-expressions
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Dec 27, 2025

⚠️ No Changeset found

Latest commit: 5c99493

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
create-t3-app Ready Ready Preview, Comment Dec 27, 2025 9:21pm

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.

1 participant