Release v4.6.0 - #4416
Conversation
- Bump version to 4.6.0 in all package.json files - Add CHANGELOG entry for v4.6.0 - Update publish workflow: replace deprecated actions/create-release with gh release create, attach dist files (less.js, less.min.js) as release assets, bump contents permission to write - Remove .github/** from paths-ignore (was preventing workflow updates) - Update CONTRIBUTING.md with detailed release documentation version: 4.6.0
- Add repository field to test-data package.json (fixes npm OIDC
provenance verification failure)
- Skip publish workflow on forks (only run on less/less.js)
- Remove duplicate require('fs') in bump-and-publish.js
- Add language specifier to markdown code block in CONTRIBUTING.md
The package is ESM-only ("type": "module"), so the README now correctly
shows `import less from 'less'` with `await` instead of CJS `require()`.
The ES6 test now verifies both promise/await and callback APIs.
version: 4.6.0
Adds index.cjs as a one-line wrapper that re-exports the ESM default. The exports field now has both import and require conditions. Adds test-cjs.cjs to verify CJS consumption alongside the existing ESM test. version: 4.6.0
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (13)
📝 WalkthroughWalkthroughThis release bumps the package to v4.6.0 with multiple improvements: adds CommonJS entry point support via new index.cjs and exports configuration, refactors the GitHub Actions publish workflow into a consolidated script-driven approach, introduces new CommonJS test coverage, updates test infrastructure, expands documentation, and bumps versions across packages. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Already merged v4.6.0 via #4415. Will create a new PR for the CJS fix, README, and workflow improvements. |
Summary
index.cjswrapper sorequire('less')works alongsideimport less from 'less', with tests for bothgithub.repositorycheck), idempotent GitHub Release creation, dist assets (less.js/less.min.js) attached to releases, squash-merge-safe version detectionrepositoryfield to@less/test-datafor npm OIDC verificationCloses
Closes #4230, closes #4186, closes #4171, closes #4151, closes #4092, closes #3990, closes #3966
Test plan
node test/test-es6.js— ESM import with promise/await and callbacknode test/test-cjs.cjs— CJS require('less') with promiseversion: 4.6.0
Summary by CodeRabbit
Release Notes - v4.6.0
New Features
Documentation
Tests
Chores