Skip to content

Improve build configuration and add Prettier formatting#401

Merged
isc-klu merged 13 commits intointersystems:masterfrom
isc-klu:update-config-pretty-ts
Apr 24, 2026
Merged

Improve build configuration and add Prettier formatting#401
isc-klu merged 13 commits intointersystems:masterfrom
isc-klu:update-config-pretty-ts

Conversation

@isc-klu
Copy link
Copy Markdown
Collaborator

@isc-klu isc-klu commented Apr 23, 2026

This PR adjusts various configuration files to improve long-term maintainability of the codebase:

  • Simplifies tsconfig.json and eslint.config.mjs by removing configuration options that are no longer necessary.
  • Adds Prettier for consistent code style checking and formatting, aligning with the sibling codebase (vscode-objectscript).

Notable changes

The majority of file changes are automated Prettier formatting and are not individually noteworthy. The meaningful changes are:

File(s) Change
eslint.config.mjs, tsconfig.json Simplified by removing unnecessary configuration
.gitignore Added tsconfig.tsbuildinfo (TypeScript incremental build artifact)
.prettierignore Created to limit Prettier to TypeScript files only
package.json Added prettier as a devDependency with its configuration; added lint and lint-fix scripts that utilize Prettier for style checking and fixing; removed globals (unused devDependency)
package-lock.json Regenerated by npm to reflect the updated package.json
client/package-lock.json, server/package-lock.json Version numbers updated by npm to match their corresponding package.json

@isc-klu isc-klu requested a review from isc-bsaviano April 23, 2026 18:43
@isc-klu isc-klu self-assigned this Apr 23, 2026
Comment thread package.json
@isc-bsaviano
Copy link
Copy Markdown
Collaborator

Thanks for doing this! You should add the new .prettierignore file to .vscodeignore.

@isc-bsaviano
Copy link
Copy Markdown
Collaborator

@isc-klu This is ready to go once you update .vscodeignore

@isc-klu
Copy link
Copy Markdown
Collaborator Author

isc-klu commented Apr 24, 2026

@isc-klu This is ready to go once you update .vscodeignore

Added. Thanks for the reminder!

@isc-klu isc-klu merged commit ef8499a into intersystems:master Apr 24, 2026
11 checks passed
@isc-klu isc-klu deleted the update-config-pretty-ts branch April 24, 2026 13:45
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.

2 participants