Skip to content

update versions and fix build issues: node, ts, eslint, typedoc, jest#76

Merged
DmitryBorisov-sm merged 21 commits into
mainfrom
update-ts-to-v5
Sep 4, 2025
Merged

update versions and fix build issues: node, ts, eslint, typedoc, jest#76
DmitryBorisov-sm merged 21 commits into
mainfrom
update-ts-to-v5

Conversation

@DmitryBorisov-sm
Copy link
Copy Markdown
Contributor

@DmitryBorisov-sm DmitryBorisov-sm commented Aug 25, 2025

Updated Node.js to the latest LTS version (22)
Updated TypeScript to the latest version (5.9)
Updated a few related libraries and their configuration files (including Jest and ESLint)
Fixed build-related issues
Fixed several vulnerabilities caused by outdated npm packages
Updated GitHub workflows to align with the Node.js update
Added the --frozen-lockfile flag to yarn run build in GitHub workflows to ensure consistent and reproducible builds
Removed package-lock.json since Yarn is used for builds in GitHub workflows
Added a developer-friendly .env.example
Fixed a few failing tests

@DmitryBorisov-sm DmitryBorisov-sm changed the title update versions: node, ts, eslint, typedoc, jest update versions and fix build issues: node, ts, eslint, typedoc, jest Aug 28, 2025
Copy link
Copy Markdown

@abhineet-orkes abhineet-orkes left a comment

Choose a reason for hiding this comment

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

lgtm

Comment thread package.json Outdated
Comment thread eslint.config.js Outdated
Comment thread tsconfig.json Outdated
Comment thread eslint.config.js Outdated
@DmitryBorisov-sm
Copy link
Copy Markdown
Contributor Author

DmitryBorisov-sm commented Aug 29, 2025

@ajlai-orkes @abhineet-orkes
Hi mates!
Please re-review the PR. I have reviewed the assembly issue from a testing perspective. I plan to add test execution during CI/CD process for different Node versions as part of a separate task (18, 20, 22), so limiting support to Node 18+ (instead of creating es5 bundle) proved to be the more rational approach - I have updated the code accordingly. I also noticed that babel was not actually used during build and in the end decided to remove it from dependencies.

I also completely rewrote the eslint config to follow modern standards with a large set of rules. Some of these rules are currently disabled, as there are more than 80 existing violations in the codebase; I will address these in a separate task.

Copy link
Copy Markdown

@KonstantinVoytovich-sm KonstantinVoytovich-sm left a comment

Choose a reason for hiding this comment

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

Overall looks good, though I would better add empty lines at the end of the files

@DmitryBorisov-sm DmitryBorisov-sm merged commit 06b31bb into main Sep 4, 2025
2 checks passed
@KonstantinVoytovich-sm KonstantinVoytovich-sm deleted the update-ts-to-v5 branch September 4, 2025 19:05
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.

4 participants