Skip to content

Add devcontainer#148

Merged
rajbos merged 8 commits intorajbos:mainfrom
UncleBats:add-devcontainer
Jan 30, 2026
Merged

Add devcontainer#148
rajbos merged 8 commits intorajbos:mainfrom
UncleBats:add-devcontainer

Conversation

@UncleBats
Copy link
Contributor

This pull request introduces several improvements to the development environment and build tooling, focusing on better cross-platform compatibility, consistent code formatting, and streamlined task definitions. The main changes include adding configuration files for Dev Containers and Git attributes, and simplifying VS Code task definitions.

Development environment enhancements:

  • Added .devcontainer/devcontainer.json to define a Dev Container setup with a TypeScript/Node.js image, essential VS Code extensions, and post-creation npm install for a consistent onboarding experience.

Cross-platform and formatting improvements:

  • Introduced .gitattributes to enforce LF line endings for text files, CRLF for Windows scripts, and proper handling of binary files, ensuring consistency across different operating systems.

Build tooling and task simplification:

  • Updated .vscode/tasks.json:
    • Simplified the "npm compile" task to use the npm task runner instead of invoking PowerShell directly.
    • Refactored "watch" and "tasks: watch-tests" tasks to use array notation for dependsOn and added a dedicated "npm: watch" background task for improved workflow. [1] [2]

Copilot AI review requested due to automatic review settings January 30, 2026 09:28
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 development container configuration and improves cross-platform compatibility by standardizing line endings and simplifying build task definitions.

Changes:

  • Adds Dev Container configuration for consistent TypeScript/Node.js development environment
  • Introduces .gitattributes to enforce platform-appropriate line endings
  • Simplifies VS Code task definitions by using npm task runner and array notation

Reviewed changes

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

File Description
.devcontainer/devcontainer.json Adds Dev Container setup with TypeScript/Node.js image and essential extensions
.gitattributes Enforces LF line endings for text files and CRLF for Windows scripts
.vscode/tasks.json Simplifies npm compile task and refactors watch tasks to use array notation

@rajbos rajbos enabled auto-merge January 30, 2026 09:43
@rajbos rajbos merged commit 003ef3b into rajbos:main Jan 30, 2026
12 checks passed
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

Comments