Skip to content

Disable Node.js setup in CI workflows - #959

Merged
Carreau merged 2 commits into
mainfrom
claude/jupyter-node-version-skip-pllkpv
Jul 25, 2026
Merged

Disable Node.js setup in CI workflows#959
Carreau merged 2 commits into
mainfrom
claude/jupyter-node-version-skip-pllkpv

Conversation

@Carreau

@Carreau Carreau commented Jul 25, 2026

Copy link
Copy Markdown
Member

Summary

This PR disables Node.js installation across all CI/CD workflows by explicitly setting node_version: "none" in the base-setup action configuration. This change optimizes CI performance by skipping unnecessary Node.js setup for Python-only projects.

Changes Made

  • Added node_version: "none" parameter to all jupyterlab/maintainer-tools/.github/actions/base-setup@v1 action calls across multiple workflows:
    • tests.yml: 6 job configurations (test, test_minimum_versions, lint, docs, test_prerelease, build_sdist, test_sdist, check_links, check_release)
    • prep-release.yml: 1 job configuration
    • publish-changelog.yml: 1 job configuration
    • publish-release.yml: 1 job configuration

Implementation Details

  • The change is consistent across all workflows, ensuring uniform behavior
  • No functional changes to the actual test/build/release logic
  • This optimization reduces CI execution time by eliminating unnecessary Node.js environment setup for a Python-focused project

https://claude.ai/code/session_01E4KrMceq44AzZCZfxt7sam

claude added 2 commits July 22, 2026 16:54
traitlets has no JavaScript, so set node_version: "none" on the
maintainer-tools base-setup action to skip installing node in the
test and release workflows.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E4KrMceq44AzZCZfxt7sam
- Use uv for base-setup's pip operations (python_package_manager: "uv pip").
- Run free-threaded 3.14t on Linux only. The free-threaded build is not in
  the runner tool cache and is expensive to provision on Windows/macOS
  (~73s base-setup vs ~23s on Linux); Linux coverage is retained.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E4KrMceq44AzZCZfxt7sam
@Carreau
Carreau merged commit a531a13 into main Jul 25, 2026
52 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