Skip to content

Rename positional argument, clean up - #35

Merged
j9t merged 4 commits into
mainfrom
cli-path-placeholder
Aug 13, 2026
Merged

Rename positional argument, clean up#35
j9t merged 4 commits into
mainfrom
cli-path-placeholder

Conversation

@j9t

@j9t j9t commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Documentation

    • Updated CLI guidance to accept files, directories, and standard input via -.
    • Clarified default directory behavior, supported CSS file handling, fix-mode output, and numeric exit statuses.
    • Documented the renamed positional path argument.
  • Chores

    • Updated the package version to 1.12.1.
    • Removed obsolete internal comments.

j9t added 3 commits August 12, 2026 21:59
Updated the CLI help output and README to rename the argument `file` to `path`, better reflecting its ability to handle both files and directories. Incremented the version to 1.12.1 in relevant metadata files.

(This commit message was AI-generated.)

Signed-off-by: Jens Oliver Meiert <jens@meiert.com>
Signed-off-by: Jens Oliver Meiert <jens@meiert.com>
Signed-off-by: Jens Oliver Meiert <jens@meiert.com>
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: ac9bc18c-e5f5-4037-a0ac-147b0640b8fc

📥 Commits

Reviewing files that changed from the base of the PR and between 36548cd and b7d3f65.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json

Walkthrough

The CLI documentation now describes multiple paths, directory scanning, STDIN input, and fix-mode output. Exit-status examples use numeric values. The package version is 1.12.1. Obsolete comments were removed or shortened.

Changes

CLI documentation release

Layer / File(s) Summary
Path and STDIN input documentation
src/cli/options.js, README.md, bin/css-dedup.js
CLI help and README text describe path inputs, directory scanning, STDIN, and --fix - output. Obsolete comments were removed or shortened.
Release and exit-status documentation
package.json, README.md, CHANGELOG.md
The package version is 1.12.1. Exit-status examples use numeric values. The changelog includes the new release entry and normalized exit-code notation.

Estimated code review effort: 2 (Simple) | ~10 minutes

Mergeability Score: ⚪ Minimal · up to b7d3f

This localized cleanup is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

Possibly related PRs

  • j9t/css-dedup#6: Updates related CLI option parsing, ignore paths, and target processing.
  • j9t/css-dedup#33: Updates related CLI positional-input help and documentation.
  • j9t/css-dedup#34: Updates related CLI positional path handling and documentation.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: renaming the positional argument and making related cleanup updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cli-path-placeholder

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.

Signed-off-by: Jens Oliver Meiert <jens@meiert.com>
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedglobals@​17.10.0 ⏵ 17.11.0100 +110086 +196 +2100

View full report

@j9t
j9t merged commit f92f3e7 into main Aug 13, 2026
6 checks passed
@j9t
j9t deleted the cli-path-placeholder branch August 13, 2026 07:14
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