Skip to content

chore(deps): remove set-blocking, is-stream, get-port#4311

Merged
JamesHenry merged 1 commit intolerna:mainfrom
AI-JamesHenry-Org:aijameshenry/remove-set-blocking-is-stream-get-port
Mar 13, 2026
Merged

chore(deps): remove set-blocking, is-stream, get-port#4311
JamesHenry merged 1 commit intolerna:mainfrom
AI-JamesHenry-Org:aijameshenry/remove-set-blocking-is-stream-get-port

Conversation

@AI-JamesHenry
Copy link
Copy Markdown
Collaborator

Note

🤖 This PR was created by @AI-JamesHenry, an AI assistant account guided and overseen by @JamesHenry.

Summary

  • set-blocking — inlined (~5 lines) with comment explaining it's effectively a no-op on POSIX but kept for Windows TTY edge case
  • is-stream — replaced with inline typeof pipe === 'function' check
  • get-port — removed (unused in production code)

Test plan

  • nx run-many -t build passes
  • nx run-many -t lint passes
  • nx run-many -t test passes
  • npm run format:check passes

- set-blocking: inlined with explanatory comment (effectively a no-op on
  POSIX, kept for Windows TTY edge case out of caution)
- is-stream: replaced with inline type check
- get-port: removed (unused in production code)
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Mar 13, 2026

View your CI Pipeline Execution ↗ for commit c2b8820

Command Status Duration Result
nx run-many --t e2e --parallel=1 ✅ Succeeded 10m 56s View ↗
nx run-many -t test --parallel=3 --ci --maxWork... ✅ Succeeded 1m 10s View ↗
nx run-many -t build --parallel=3 ✅ Succeeded 46s View ↗
nx run-many -t lint --parallel=3 ✅ Succeeded 44s View ↗
nx run integration:integration --ci --maxWorkers=2 ✅ Succeeded 12s View ↗
nx run-many -t build ✅ Succeeded <1s View ↗
nx-cloud record -- npx nx format:check ✅ Succeeded 2s View ↗
nx run-many -t test --parallel=3 --ci --maxWork... ✅ Succeeded 3m 3s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-13 11:21:11 UTC

@JamesHenry JamesHenry marked this pull request as ready for review March 13, 2026 11:22
@JamesHenry JamesHenry merged commit ebf6729 into lerna:main Mar 13, 2026
15 checks passed
@JamesHenry JamesHenry deleted the aijameshenry/remove-set-blocking-is-stream-get-port branch March 13, 2026 11:27
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