-
Notifications
You must be signed in to change notification settings - Fork 1
feat(linux): Add ffmpeg #159
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
Conversation
CLose #153 Signed-off-by: unknowIfGuestInDream <liang.tang.cx@gmail.com>
Reviewer's GuideThis pull request adds a link to the new ffmpeg installation guide in the Linux documentation sidebar. File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Their most recently public accepted PR is: unknowIfGuestInDream/javafxTool#2128 |
Caution Review failedThe pull request is closed. WalkthroughA new documentation entry for installing FFmpeg on Tencent CentOS 8 has been added. The sidebar navigation was updated to include this new FFmpeg installation guide directly after the Oracle installation entry. The new documentation describes the steps to enable necessary repositories and install FFmpeg using the Negativo17 repository, followed by a verification step. No code or public API changes were made. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Sidebar
participant FFmpegDoc
User->>Sidebar: Selects "安装ffmpeg"
Sidebar->>FFmpegDoc: Opens FFmpeg installation guide
FFmpegDoc->>User: Displays installation steps and verification
Assessment against linked issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Hey @unknowIfGuestInDream - I've reviewed your changes - here's some feedback:
- This PR's title mentions adding ffmpeg, but the code changes only involve adding a newline to a sidebar file.
- Please fill out the 'Proposed Changes' section in the description to explain the changes.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
CLose #153 Signed-off-by: unknowIfGuestInDream <liang.tang.cx@gmail.com>
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue or pull request. |
CLose #153
Fixes #
Proposed Changes
Readiness Checklist
Author/Contributor
Reviewing Maintainer
enhancement
,bug
,documentation
orstyle
Summary by Sourcery
Documentation:
Summary by CodeRabbit