Skip to content

Conversation

@ScottyPoi
Copy link

Upgrade and Modernize blockapps-rest Development Environment

Overview

This PR modernizes the development environment and tooling for the blockapps-rest component. The changes focus on upgrading Node.js and its ecosystem tools while improving the development experience and code quality assurance.

Changes

Runtime and Core Dependencies

  • Upgraded Node.js to LTS version 22.16.0
  • Updated @types/node to match the new Node.js version
  • Updated and fixed axios implementation

Build System Modernization

  • Replaced gulp with tsx for a more modern build approach
  • Updated TypeScript configuration
  • Created separate config files for JavaScript modules (JSM) and CommonJS (CJS)
  • Added file extensions to import paths for better module resolution

Code Quality Tools

  • Updated ESLint and Prettier configurations
  • Implemented comprehensive linting and formatting fixes
  • Updated configuration path handling

Documentation

  • Added DEVELOPMENT.md with detailed guidelines for code quality tools and workflows

Impact

These changes modernize our development environment and improve code quality by:

  1. Leveraging the latest Node.js LTS features and security improvements
  2. Streamlining the build process with modern tools
  3. Enhancing code quality through updated linting and formatting standards
  4. Providing better TypeScript support and type safety

Copy link
Author

@ScottyPoi ScottyPoi left a comment

Choose a reason for hiding this comment

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

Note:
Some of the linting and formatting rules were set to warn rather than error simply to limit the scope of this PR. For example no-explicit-any is a prefered rule that I would set to error, but that would require many actual code changes to accomplish. These type of changes should be made in a future dedicated PR.

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