Skip to content

Conversation

@LuudJanssen
Copy link
Member

  • Adds NestJS / React Router configs for ESLint and Typescript
  • Adds the new configs to the docs
  • Disables some annoying ESLint rules
  • Small documentation fixes
  • Dependency updates

@changeset-bot
Copy link

changeset-bot bot commented Jan 29, 2026

🦋 Changeset detected

Latest commit: d97ae13

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@incentro-ic/config-typescript Minor
@incentro-ic/config-prettier Patch
@incentro-ic/config-cspell Patch
@incentro-ic/config-eslint Minor

Not sure what this means? Click here to learn what changesets are.

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds TypeScript and ESLint configurations for NestJS and React Router projects, while also updating dependencies and making minor improvements to existing configurations.

Changes:

  • Adds new TypeScript and ESLint configurations for NestJS and React Router applications
  • Disables several ESLint rules that were causing friction (unused variables with underscore prefix, void return promises, arrow function scoping)
  • Updates multiple dependencies across packages (prettier, @types/node, eslint plugins, etc.)

Reviewed changes

Copilot reviewed 25 out of 26 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/typescript/src/package-node/tsconfig.json Standardizes casing of "target" property value
packages/typescript/src/app-react-router/tsconfig.json Adds new TypeScript config for React Router apps
packages/typescript/src/app-nest/tsconfig.json Adds new TypeScript config for NestJS apps
packages/typescript/package.json Exports new configs and updates dependencies
packages/typescript/README.md Documents the new NestJS and React Router configurations
packages/prettier/src/plugins/package-json.ts Makes preprocess function async to match API requirements
packages/prettier/package.json Updates dependencies
packages/prettier/README.md Clarifies package.json plugin description
packages/eslint/src/react-router.ts Adds new ESLint config for React Router projects
packages/eslint/src/partials/typescript.ts Adds new ESLint rule configurations and reformats comments
packages/eslint/src/partials/base.ts Disables arrow function checking in consistent-function-scoping rule
packages/eslint/src/nest.ts Adds new ESLint config for NestJS projects
packages/eslint/package.json Exports new configs and updates dependencies
packages/eslint/README.md Documents the new NestJS and React Router configurations
packages/cspell/package.json Updates dependencies
packages/cspell/README.md Adds documentation for Dutch language support
package.json Updates root dependencies and package manager version
apps/docs/package.json Updates documentation app dependencies
.nvmrc Updates Node.js version
.changeset/*.md Adds changeset files for version tracking

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 25 out of 26 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

LuudJanssen and others added 2 commits January 30, 2026 08:04
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@LuudJanssen LuudJanssen merged commit 271989e into main Jan 30, 2026
2 checks passed
@LuudJanssen LuudJanssen deleted the feature/nest-and-react-router-configs branch January 30, 2026 11:37
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