-
Notifications
You must be signed in to change notification settings - Fork 54
Update website, remove blog, added snow #180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for pelica ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Warning Rate limit exceeded@notAreYouScared has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 1 minutes and 39 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
WalkthroughThis PR removes multiple blog posts and several author entries, strips blog-related UI and plugin configuration from Docusaurus, relocates markdown link-check settings into Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (9)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yamlstatic/img/blog/1.pngis excluded by!**/*.pngstatic/img/blog/2.pngis excluded by!**/*.pngstatic/img/blog/3.pngis excluded by!**/*.pngstatic/img/blog/4.pngis excluded by!**/*.pngstatic/img/blog/5.pngis excluded by!**/*.pngstatic/img/homepage/sponsor/surfhosting.pngis excluded by!**/*.pngstatic/img/homepage/sponsor/vultric.pngis excluded by!**/*.pngstatic/js/snowflake.min.jsis excluded by!**/*.min.js
📒 Files selected for processing (14)
blog/2024-04-12-announcement.mdx(0 hunks)blog/2024-04-15-head-start.mdx(0 hunks)blog/2024-04-26-license-questions.mdx(0 hunks)blog/2024-05-03-translations-begin.mdx(0 hunks)blog/2024-05-04-agpl-relicensing.mdx(0 hunks)blog/authors.yml(0 hunks)docusaurus.config.ts(2 hunks)package.json(1 hunks)pnpm-workspace.yaml(1 hunks)src/components/HomepageFeatures/look.tsx(2 hunks)src/components/HomepageFeatures/sponsors.tsx(1 hunks)src/components/HomepageFeatures/what.tsx(1 hunks)src/components/HomepageFeatures/why.tsx(1 hunks)src/pages/index.tsx(1 hunks)
💤 Files with no reviewable changes (6)
- blog/2024-05-04-agpl-relicensing.mdx
- blog/2024-04-12-announcement.mdx
- blog/2024-04-26-license-questions.mdx
- blog/authors.yml
- blog/2024-05-03-translations-begin.mdx
- blog/2024-04-15-head-start.mdx
🧰 Additional context used
🪛 Biome (2.1.2)
src/components/HomepageFeatures/look.tsx
[error] 43-43: Avoid passing children using a prop
The canonical way to pass children in React is to use JSX elements
(lint/correctness/noChildrenProp)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: Redirect rules
- GitHub Check: Header rules
- GitHub Check: Pages changed
🔇 Additional comments (10)
src/components/HomepageFeatures/what.tsx (1)
1-1: LGTM! Explicit JSX type import added.The addition of the JSX type import aligns with the explicit
JSX.Elementreturn type used in the exported component function.src/pages/index.tsx (1)
1-1: LGTM! Explicit JSX type import added.Consistent with the explicit
JSX.Elementreturn type annotation on the exportedHomecomponent.src/components/HomepageFeatures/why.tsx (1)
1-1: LGTM! Explicit JSX type import added.The JSX type import is appropriate for the explicit
JSX.Elementtyping used in both the component return type and theFeatureItemtype definition.src/components/HomepageFeatures/sponsors.tsx (1)
1-1: LGTM! Explicit JSX type import added.Consistent with the explicit
JSX.Elementreturn type on the exportedHomepageSponsorcomponent.src/components/HomepageFeatures/look.tsx (1)
1-1: LGTM! Explicit JSX type import added.Consistent with the explicit
JSX.Elementreturn type on the exported component.package.json (3)
45-52: LGTM on devDependencies upgrades.The development dependency upgrades appear reasonable:
- Docusaurus tooling packages align with the main version upgrade
- TypeScript 5.9.3 is a stable recent release
- Tailwind and autoprefixer are patch/minor version bumps
37-37: LGTM on PostCSS import addition.Adding
postcss-importis a standard practice for handling CSS@importstatements in PostCSS/Tailwind workflows. This aligns well with the removal of PostHog and the existing Tailwind setup.
39-41: Docusaurus 3.9 officially supports React 19—no compatibility verification needed.Docusaurus is now compatible with React 19, with all newly initialized sites using React 19 by default. The React 19 upgrade is safe and supported by Docusaurus 3.9.2.
docusaurus.config.ts (2)
13-15: LGTM on markdown link checking.Adding broken markdown link detection is a good practice that will help catch documentation issues early. Using
"warn"instead of"throw"is a reasonable choice that won't break the build but will surface issues.
132-132: File exists and is properly sourced – no action required.The
snowflake.min.jsfile is present in the repository atstatic/js/snowflake.min.js(3.5KB). The file includes proper attribution crediting https://github.com/nextapps-de/snowflake with Apache License 2.0, confirming it's a legitimate open-source library. The path format./static/js/snowflake.min.jsis correct for DocusaurusclientModulesconfiguration. No concerns.Likely an incorrect or invalid review comment.

What's wrong with a little bit of snow? NOTHING!
This pull request includes a major cleanup and update of the project’s blog and configuration files, as well as dependency upgrades and minor code improvements. The most significant changes are the removal of all blog post content and author data, updates to Docusaurus configuration and dependencies, and some minor code adjustments in React components.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.