Skip to content

allow text-wrap#6

Open
AlbertSmit wants to merge 1 commit into
mainfrom
allow-text-wrap
Open

allow text-wrap#6
AlbertSmit wants to merge 1 commit into
mainfrom
allow-text-wrap

Conversation

@AlbertSmit
Copy link
Copy Markdown
Contributor

@AlbertSmit AlbertSmit commented Oct 30, 2025

Summary by CodeRabbit

  • Chores
    • Updated linting configuration to support additional CSS properties in style validation.

@AlbertSmit AlbertSmit self-assigned this Oct 30, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 30, 2025

Walkthrough

The .stylelintrc configuration file was updated to extend the property-no-unknown rule by adding an ignoreProperties array that includes the text-wrap property, while retaining the existing ignoreSelectors configuration for :export.

Changes

Cohort / File(s) Summary
Stylelint configuration
.stylelintrc
Extended property-no-unknown rule with new ignoreProperties list to ignore text-wrap property

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The PR title "allow text-wrap" is directly and fully related to the primary change in the changeset. The raw summary indicates that the modification extends the property-no-unknown rule by adding text-wrap to the ignoreProperties list in .stylelintrc, which effectively allows this property. The title is concise, clear, and specific enough for a teammate reviewing the commit history to understand that this PR enables support for the text-wrap property in the stylelint configuration.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch allow-text-wrap

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2ab5d11 and ebb4620.

📒 Files selected for processing (1)
  • .stylelintrc (1 hunks)
🔇 Additional comments (1)
.stylelintrc (1)

152-160: Configuration change looks good!

The modification correctly extends the property-no-unknown rule to ignore the text-wrap property while preserving the existing ignoreSelectors configuration. The syntax and structure are valid for stylelint, and this aligns with the PR objective.

Just to confirm: please verify that your stylelint version supports the ignoreProperties option for the property-no-unknown rule (this has been standard for several years, but good to confirm your setup).


Comment @coderabbitai help to get the list of available commands and usage tips.

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