Skip to content

ci: add GitHub Actions workflow to run tests on PRs#124

Closed
WuKongAI-CMU wants to merge 1 commit intoNVIDIA:mainfrom
WuKongAI-CMU:ci/add-test-workflow
Closed

ci: add GitHub Actions workflow to run tests on PRs#124
WuKongAI-CMU wants to merge 1 commit intoNVIDIA:mainfrom
WuKongAI-CMU:ci/add-test-workflow

Conversation

@WuKongAI-CMU
Copy link
Copy Markdown
Contributor

Summary

  • Adds .github/workflows/test.yml that runs on every push to main and every pull request
  • Installs dependencies, builds the TypeScript plugin, and runs node --test test/*.test.js
  • Tests against Node.js 20 and 22 (matching the project's minimum supported version)

Currently there is no CI in the repository — reviewers must run tests locally. This workflow gives maintainers automated pass/fail feedback on every PR.

Test plan

  • Workflow triggers on this PR itself (check the Actions tab)
  • No changes to production code

Runs the test suite (`node --test test/*.test.js`) and TypeScript build
on every push to main and every pull request. Tests against Node.js 20
and 22 to match the project's minimum supported version.

Signed-off-by: Peter Tam <nicholaslwk@gmail.com>
Signed-off-by: peteryuqin <peter.yuqin@gmail.com>
@WuKongAI-CMU
Copy link
Copy Markdown
Contributor Author

Closing to reduce my open PR count below the repo policy limit and refocus on a smaller set of higher-signal changes. I can revive this branch later if it becomes the right path again.

mafueee pushed a commit to mafueee/NemoClaw that referenced this pull request Mar 28, 2026
…#124)

* initial doc filling

* improvements

* stage provided get started, and add clean tutorials

* pull in Kirit's content and polish

* improve observability

* moving pieces

* move TOC around

* drop support matrix from concepts

* fix links

* minor fixes and fix badges

* minor fixes

* incorporate missed content

* minor improvements

* clean up

* run dori style guide review

* clean up

* updates impacting docs

* incorporate feedback

* minor fix

* some edits

* enterprise structure

* update cards

* improve

* add some emojis

* improve landing page with animated getting started code

* fix the animated code

* small improvements

* refresh content based on PR 156 and 158

* README as the source of truth for quickstart

* update README

* run edits

* change to the new prod name, text only, code swipe later

* add Home

* incorporate dev feedback on README

* krit's edits

* edit and improve index pages

* fix build

* revert README

* revert quickstart to not pull from README
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