Skip to content

feat: node js image added#30

Merged
tareq1988 merged 1 commit intomainfrom
node
Jan 24, 2026
Merged

feat: node js image added#30
tareq1988 merged 1 commit intomainfrom
node

Conversation

@tareq1988
Copy link
Contributor

@tareq1988 tareq1988 commented Jan 24, 2026

Summary by CodeRabbit

Release Notes

  • Documentation

    • Updated PHP testing instructions in README
    • Added Node.js (Alpine) documentation including supported versions (18, 20, 22, 24, 25) and usage guidance
  • Tests

    • Added Node.js structure tests for validating tools (Node, Yarn, PNPM, Vite, Git)
  • Chores

    • Added Dockerfile and build script for multi-architecture Node.js container builds

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 24, 2026

Walkthrough

This PR introduces a new Node.js Docker image infrastructure with Alpine-based configurations supporting versions 18, 20, 22, 24, and 25. It includes a Dockerfile, multi-architecture build script, structure tests for validation, and documentation updates.

Changes

Cohort / File(s) Change Summary
Documentation
README.MD
Updated PHP testing instruction reference; added comprehensive Node.js (Alpine) section covering supported versions, image acquisition, usage patterns (one-off and docker-compose), and testing procedures with build script details.
Node.js Docker Configuration
node/Dockerfile
New Dockerfile using Alpine base with Node.js, installs git and bash system dependencies, provisions global npm packages (yarn, pnpm, vite), and sets node as default command.
Build Orchestration
node/build.sh
New shell script orchestrating multi-architecture Docker builds (linux/amd64, linux/arm64) across Node versions 18, 20, 22, 24, 25, with BuildKit enabled, image tagging, and registry push.
Testing & Validation
node-structure-test.yaml
New structure test configuration (schema 2.0.0) validating presence and output of Node, Yarn, PNPM, Vite, and Git tools via command execution.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A fluffy build script hops along,
With Node and Alpine, strong and song,
Multi-arch dreams in docker's sight,
Yarn and Vite, all burning bright!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: node js image added' clearly summarizes the main objective of the pull request, which is to add a Node.js Docker image with supporting files and documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch node

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tareq1988 tareq1988 merged commit 1259f0a into main Jan 24, 2026
1 of 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.

1 participant