Skip to content

[chores] Website maintenance#161

Merged
nemesifier merged 3 commits intomasterfrom
maintenance
Jan 8, 2026
Merged

[chores] Website maintenance#161
nemesifier merged 3 commits intomasterfrom
maintenance

Conversation

@nemesifier
Copy link
Copy Markdown
Member

  • Hide coderabbitai reviews in activity feed
  • Fixed deprecation warning due to SASS darken()
  • Fixed warning due to postcss.config.js being in ESM

@nemesifier nemesifier self-assigned this Jan 8, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 8, 2026

Walkthrough

The pull request comprises four independent changes across different file types: a Sass color function refactor replacing darken() with color.adjust() while importing the color module; a conditional guard in the activity feed to clear content when the actor is the CodeRabbit bot; a module system conversion from ES modules to CommonJS in the PostCSS configuration; and a version number bump in the page-activity HTML template from v=1 to v=2.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title '[chores] Website maintenance' is vague and generic, using non-descriptive terms that don't convey what the actual changes entail. Consider a more specific title like '[chores] Hide CodeRabbit reviews and fix deprecation warnings' to better reflect the actual changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The description clearly relates to the changeset, listing three specific fixes: hiding CodeRabbit reviews, fixing SASS deprecation warning, and fixing PostCSS ESM warning.

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

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 418dc2b and 251a901.

📒 Files selected for processing (4)
  • css/theme.scss
  • js/activity/feed.js
  • postcss.config.js
  • theme/templates/page-activity.html
🔇 Additional comments (5)
theme/templates/page-activity.html (1)

3-3: LGTM! Appropriate cache busting for the activity feed changes.

The version bump ensures clients fetch the updated JavaScript that includes the CodeRabbit bot filtering logic.

css/theme.scss (2)

78-78: LGTM! Proper Sass module import for modern color functions.

The @use "sass:color" import enables the use of the modern color.adjust() function, replacing the deprecated darken() function.


482-482: LGTM! Correct migration from deprecated darken() to color.adjust().

The conversion from darken($orange-light, 5%) to color.adjust($orange-light, $lightness: -5%) is functionally equivalent and resolves the Sass deprecation warning mentioned in the PR description.

postcss.config.js (1)

1-7: LGTM! Correct conversion from ES modules to CommonJS.

The conversion from import/export to require()/module.exports resolves the PostCSS ESM warning mentioned in the PR description while maintaining identical functionality.

js/activity/feed.js (1)

297-300: Verify: Behavior clears content but retains activity entry.

The current implementation clears the content field for CodeRabbit bot activities, which means:

  • The activity card will still appear in the feed
  • The header/title and event type will be visible
  • Only the detailed review content will be hidden

If the intent is to completely hide CodeRabbit activities from the feed, consider returning early instead:

Alternative: Skip rendering entirely
 
 // truncate AI reviews to avoid flooding the feed
 if (field.actor.login === "coderabbitai[bot]") {
-  content = "";
+  return; // Skip rendering this activity entirely
 }

The current approach is reasonable if you want to show that CodeRabbit reviewed something without displaying the lengthy review details. Please confirm this matches your intended behavior.


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

@nemesifier nemesifier merged commit 251a901 into master Jan 8, 2026
3 checks passed
@nemesifier nemesifier deleted the maintenance branch February 4, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant