Update website, remove blog, added snow#180
Conversation
✅ Deploy Preview for pelica ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Caution Review failedThe pull request is closed. 📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughRemoves five blog posts and five author entries, strips blog UI/plugin references from Docusaurus, moves markdown link-check to Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Pre-merge checks and finishing touches❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
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.
|
I think you can delete that FAQ: Check other FAQs, I think some of them need to be changed or removed directly. |
Comment out the snowflake.min.js script in the config.
✅ Deploy Preview for pelica ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
2 participants
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