Skip to content

chore: add typescript configuration files to scripts directory#4093

Merged
sgaczol merged 2 commits into
mainfrom
@sgaczol/scripts-tsconfig
May 25, 2026
Merged

chore: add typescript configuration files to scripts directory#4093
sgaczol merged 2 commits into
mainfrom
@sgaczol/scripts-tsconfig

Conversation

@sgaczol
Copy link
Copy Markdown
Collaborator

@sgaczol sgaczol commented May 25, 2026

Description

Adding tsconfig.json and .eslintrc.js to scripts directory is necessary in order for the new script, introduced in #4087, to work properly.

Changes

  • add tsconfig.json and .eslintrc.js to scripts directory

Before & after - visual documentation

N/A

Test plan

N/A

Checklist

  • Included code example that can be used to test this change.
  • For visual changes, included screenshots / GIFs / recordings documenting the change.
  • For API changes, updated relevant public types.
  • Ensured that CI passes

Copy link
Copy Markdown
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

Adds TypeScript and ESLint configuration files under scripts/ to support running/linting TypeScript-based utility scripts from that directory (notably the checklist-generation script referenced in #4087).

Changes:

  • Add scripts/tsconfig.json extending the repo tsconfig.json and defining path aliases for script compilation.
  • Add scripts/.eslintrc.js to provide an ESLint root config for files under scripts/.

Reviewed changes

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

File Description
scripts/tsconfig.json Adds a scripts-local tsconfig with paths mappings for module resolution.
scripts/.eslintrc.js Adds a scripts-local ESLint root configuration.

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

Comment thread scripts/tsconfig.json Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@kkafar kkafar left a comment

Choose a reason for hiding this comment

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

LGTM

@sgaczol sgaczol merged commit 8b26c05 into main May 25, 2026
3 checks passed
@sgaczol sgaczol deleted the @sgaczol/scripts-tsconfig branch May 25, 2026 12:27
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