Skip to content

Branding-changes#2

Merged
Graison-P merged 1 commit intomasterfrom
branding-changes
Oct 15, 2025
Merged

Branding-changes#2
Graison-P merged 1 commit intomasterfrom
branding-changes

Conversation

@supervoidcoder
Copy link
Member

just simple branding chnages and stuff

Copy link
Member Author

@supervoidcoder supervoidcoder left a comment

Choose a reason for hiding this comment

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

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Oct 7, 2025

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • development/homepage-template.ejs
⛔ Files ignored due to path filters (1)
  • website/favicon.ico is excluded by !**/*.ico

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch branding-changes

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.

@supervoidcoder
Copy link
Member Author

@coderabbitai review

pls review

@coderabbitai
Copy link

coderabbitai bot commented Oct 7, 2025

@supervoidcoder: I'll review the changes in this PR for you.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@supervoidcoder
Copy link
Member Author

@coderabbitai why would it have no reviewable changes i changed it so much

@coderabbitai
Copy link

coderabbitai bot commented Oct 7, 2025

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get PR details including files changed, commits, and diff
echo "=== PR Details ==="
gh pr view 2 --json number,title,state,additions,deletions,changedFiles,commits,files

echo -e "\n=== PR Diff ==="
gh pr diff 2

echo -e "\n=== Commit List ==="
gh pr view 2 --json commits --jq '.commits[] | "\(.oid[0:7]) \(.messageHeadline)"'

echo -e "\n=== Files Changed ==="
gh pr view 2 --json files --jq '.files[] | "\(.path) (+\(.additions)/-\(.deletions))"'

Length of output: 466


🏁 Script executed:

#!/bin/bash
# Get PR details from the correct repository
REPO="OmniBlocks/extensions"

echo "=== PR Details ==="
gh pr view 2 -R "$REPO" --json number,title,state,additions,deletions,changedFiles,commits,files

echo -e "\n=== PR Diff ==="
gh pr diff 2 -R "$REPO"

echo -e "\n=== Commit List ==="
gh pr view 2 -R "$REPO" --json commits --jq '.commits[] | "\(.oid[0:7]) \(.messageHeadline)"'

echo -e "\n=== Files Changed ==="
gh pr view 2 -R "$REPO" --json files --jq '.files[] | "\(.path) (+\(.additions)/-\(.deletions))"'

Unrecoverable error: Sandbox execution failed: max_buffer, terminated


🏁 Script executed:

#!/bin/bash
# Get PR info without the full diff to avoid buffer overflow
REPO="OmniBlocks/extensions"

echo "=== Basic PR Info ==="
gh pr view 2 -R "$REPO" --json number,title,state,additions,deletions,changedFiles

echo -e "\n=== Files Changed (names only) ==="
gh pr view 2 -R "$REPO" --json files --jq '.files[] | .path'

echo -e "\n=== File Count by Extension ==="
gh pr view 2 -R "$REPO" --json files --jq '.files[] | .path' | sed 's/.*\.//' | sort | uniq -c | sort -rn

echo -e "\n=== Check for common excluded file patterns ==="
gh pr view 2 -R "$REPO" --json files --jq '.files[] | .path' | grep -E '(package-lock\.json|yarn\.lock|\.min\.js|\.map|dist/|build/|node_modules/)' || echo "No common excluded patterns found"

Length of output: 354


🏁 Script executed:

#!/bin/bash
REPO="OmniBlocks/extensions"

echo "=== Checking .ejs file diff (first 100 lines) ==="
gh pr diff 2 -R "$REPO" -- development/homepage-template.ejs | head -100

echo -e "\n=== File type check ==="
gh pr view 2 -R "$REPO" --json files --jq '.files[] | "\(.path): additions=\(.additions), deletions=\(.deletions)"'

echo -e "\n=== Checking if favicon is binary ==="
file website/favicon.ico 2>/dev/null || echo "File check not available in sandbox, but .ico files are binary image files"

Length of output: 334


🏁 Script executed:

#!/bin/bash
REPO="OmniBlocks/extensions"

echo "=== Checking repository configuration for review exclusions ==="
# Check if there's a .coderabbit.yaml or similar config
gh api "/repos/$REPO/contents/.coderabbit.yaml" 2>/dev/null | jq -r '.content' | base64 -d 2>/dev/null || echo "No .coderabbit.yaml found"

gh api "/repos/$REPO/contents/.coderabbit.yml" 2>/dev/null | jq -r '.content' | base64 -d 2>/dev/null || echo "No .coderabbit.yml found"

echo -e "\n=== Get partial diff of .ejs file ==="
gh pr diff 2 -R "$REPO" 2>&1 | grep -A 50 "homepage-template.ejs" | head -60

echo -e "\n=== Summary ==="
echo "Files changed:"
echo "1. development/homepage-template.ejs (+125/-43 lines) - EJS template file"
echo "2. website/favicon.ico (binary) - Image file"

Length of output: 2122


🏁 Script executed:

#!/bin/bash
REPO="OmniBlocks/extensions"

echo "=== Checking all PR comments ==="
gh pr view 2 -R "$REPO" --json comments --jq '.comments[] | "[\(.author.login)] \(.body[0:200])"'

echo -e "\n=== Checking review comments ==="
gh pr view 2 -R "$REPO" --json reviews --jq '.reviews[] | "[\(.author.login)] State: \(.state) - \(.body[0:200])"'

echo -e "\n=== Checking PR timeline for review events ==="
gh api "/repos/$REPO/pulls/2/reviews" --jq '.[] | "[\(.user.login)] State: \(.state) - Submitted: \(.submitted_at)"'

Length of output: 817


@supervoidcoder: I can see what happened! Your PR does have changes (125 additions, 43 deletions across 2 files), but they're in files that CodeRabbit doesn't review by default:

Files changed:

  1. website/favicon.ico - This is a binary image file. CodeRabbit doesn't review binary files like images, which is why it shows 0 additions/deletions.

  2. development/homepage-template.ejs - This file has all your branding changes (TurboWarp → OmniBlocks, color scheme updates from #56b2ff to #59C0C0, etc.). However, files in the development/ directory and/or .ejs template files appear to be excluded from review by default.

Here's what I can see from your changes in the template file:

  • Title: "TurboWarp Extension Gallery" → "OmniBlocks Extension Gallery"
  • Description: references to TurboWarp → OmniBlocks
  • Color scheme updates (link colors, warning boxes, etc.)
  • Theme color and various styling changes

To include these files in future reviews, you can:

  1. Configure CodeRabbit using the .coderabbit.yaml file to include the development/ directory or .ejs files
  2. Check the path filters documentation for more details on configuring which paths to review

Would you like me to help you configure CodeRabbit to review these types of files, or do you prefer to keep them excluded from automated reviews?

@Graison-P Graison-P marked this pull request as ready for review October 15, 2025 13:42
@Graison-P Graison-P merged commit b1f73cb into master Oct 15, 2025
4 of 5 checks passed
@supervoidcoder
Copy link
Member Author

why you do nthat it was marked as draft for a reason, only merge ready prs.... Now I'm gonna have to revert all this...

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.

2 participants