Skip to content

Fix/subword config language overridable#2734

Open
whitphx wants to merge 2 commits intomainfrom
fix/subword-config-language-overridable
Open

Fix/subword config language overridable#2734
whitphx wants to merge 2 commits intomainfrom
fix/subword-config-language-overridable

Conversation

@whitphx
Copy link
Owner

@whitphx whitphx commented Feb 5, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Made the subword mode configuration setting overridable at the language level, allowing for more granular control over this feature across different programming languages.

Copilot AI review requested due to automatic review settings February 5, 2026 10:47
@changeset-bot
Copy link

changeset-bot bot commented Feb 5, 2026

🦋 Changeset detected

Latest commit: 2078986

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
emacs-mcx Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

A changeset file documents a patch release for emacs-mcx that adds a scope: "language-overridable" property to the emacs-mcx.subwordMode configuration option in package.json, enabling language-specific configuration overrides.

Changes

Cohort / File(s) Summary
Release Metadata
.changeset/sixty-beers-flow.md
New changeset file documenting a patch release for emacs-mcx with configuration scope enhancement.
Configuration Schema
package.json
Updated emacs-mcx.subwordMode configuration property to include scope: "language-overridable" for language-specific override support.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Config options now dance with grace,
Language-wide, each setting finds its place,
With "overridable" scope so fine,
Subword mode will sweetly align! 🌿

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: making the emacs-mcx.subwordMode configuration language-overridable, which is the core modification in both the changeset and package.json.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/subword-config-language-overridable

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 and usage tips.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the configuration scope of emacs-mcx.subwordMode to make it language-overridable, allowing users to set different values for different programming languages.

Changes:

  • Added "scope": "language-overridable" to the emacs-mcx.subwordMode configuration in package.json
  • Added a changeset documenting this fix

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Added "scope": "language-overridable" to the emacs-mcx.subwordMode configuration property to enable per-language configuration overrides
.changeset/sixty-beers-flow.md Added changeset documenting the configuration scope fix as a patch-level change

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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