Skip to content

chore: add root test:integration script - #198

Merged
LadyBluenotes merged 2 commits into
mainfrom
integration-tests
Sep 13, 2026
Merged

LadyBluenotes merged 2 commits into
mainfrom
integration-tests

Conversation

@LadyBluenotes

@LadyBluenotes LadyBluenotes commented Jul 9, 2026

Copy link
Copy Markdown
Member

Stack (merge in order): #198#287#288#289#290#259.

Adds the root test:integration script (nx affected --targets=test:integration), matching the existing test:lib and test:types entry points. Contributors can run affected integration tests directly from the repository root. The branch includes the current main fixes for CI dependency checks.

Summary by CodeRabbit

  • Tests
    • Added a package script for running affected integration tests.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: d95ac777-0fca-4916-b1d1-945abc043470

📥 Commits

Reviewing files that changed from the base of the PR and between 73402c1 and fa22fb7.

📒 Files selected for processing (1)
  • package.json

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The package scripts now include test:integration, which runs nx affected --targets=test:integration.

Changes

Integration test command

Layer / File(s) Summary
Add integration test script
package.json
Adds the test:integration script for affected Nx integration tests.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to fa22f

The root integration-test command is configured consistently with the repository's existing Nx targets and is ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly and concisely describes the main change: adding the root test:integration script.
Description check ✅ Passed The description clearly explains the change, command, motivation, and development-only scope. It does not use the template headings or include the checklist items, but the core required information is…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch integration-tests

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.

@LadyBluenotes
LadyBluenotes requested a review from beaussan July 9, 2026 02:40
@nx-cloud

nx-cloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit fa22fb7

Command Status Duration Result
nx affected --targets=test:eslint,test:sherif,t... ✅ Succeeded 46s View ↗
nx run-many --targets=build ✅ Succeeded 3s View ↗

☁️ Nx Cloud last updated this comment at 2026-09-13 03:57:24 UTC

@pkg-pr-new

pkg-pr-new Bot commented Jul 9, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@tanstack/intent@198

commit: fa22fb7

@LadyBluenotes LadyBluenotes changed the title ci: run integration tests in PR checks chore: add root test:integration script Sep 11, 2026
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