-
Notifications
You must be signed in to change notification settings - Fork 59
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
feat: support linglong(玲珑) #187
Conversation
审核指南由 Sourcery 提供此拉取请求增加了对 Linglong 安装方法的支持,增强了路径处理和补丁逻辑,改进了日志记录,并重构了安装脚本以提高清晰度和效率。 Linglong QQ 安装过程的序列图sequenceDiagram
participant User
participant Script
participant LingLong
participant QQ
User->>Script: 运行安装脚本
Script->>LingLong: 检查 ll-cli 命令
LingLong-->>Script: 命令存在
Script->>LingLong: 获取 QQ 版本信息
LingLong-->>Script: 返回版本
Script->>Script: 设置 QQ_PATH 和路径
Script->>Script: 安装 LiteLoaderQQNT
Script->>Script: 补丁资源
Script->>Script: 安装插件商店
Script-->>User: 安装完成
安装路径检测和补丁的状态图stateDiagram-v2
[*] --> CheckInstallMethod
CheckInstallMethod --> AUR: 是 AUR
CheckInstallMethod --> Flatpak: 是 Flatpak
CheckInstallMethod --> Linglong: 是 Linglong
CheckInstallMethod --> Traditional: 是传统方法
AUR --> PathDetection
Flatpak --> PathDetection
Linglong --> PathDetection
Traditional --> PathDetection
PathDetection --> AlreadyPatched: 路径在 patched_paths 中
PathDetection --> NeedsPatching: 路径未补丁
NeedsPatching --> InstallAndPatch
InstallAndPatch --> [*]
AlreadyPatched --> [*]
文件级别更改
针对关联问题的评估
提示和命令与 Sourcery 互动
自定义您的体验访问您的仪表板以:
获取帮助Original review guide in EnglishReviewer's Guide by SourceryThis pull request adds support for the Linglong installation method, enhances path handling and patching logic, improves logging, and refactors the installation script for better clarity and efficiency. Sequence diagram for Linglong QQ installation processsequenceDiagram
participant User
participant Script
participant LingLong
participant QQ
User->>Script: Run install script
Script->>LingLong: Check ll-cli command
LingLong-->>Script: Command exists
Script->>LingLong: Get QQ version info
LingLong-->>Script: Return version
Script->>Script: Set QQ_PATH and paths
Script->>Script: Install LiteLoaderQQNT
Script->>Script: Patch resources
Script->>Script: Install plugin store
Script-->>User: Installation complete
State diagram for installation path detection and patchingstateDiagram-v2
[*] --> CheckInstallMethod
CheckInstallMethod --> AUR: is AUR
CheckInstallMethod --> Flatpak: is Flatpak
CheckInstallMethod --> Linglong: is Linglong
CheckInstallMethod --> Traditional: is Traditional
AUR --> PathDetection
Flatpak --> PathDetection
Linglong --> PathDetection
Traditional --> PathDetection
PathDetection --> AlreadyPatched: path in patched_paths
PathDetection --> NeedsPatching: path not patched
NeedsPatching --> InstallAndPatch
InstallAndPatch --> [*]
AlreadyPatched --> [*]
File-Level Changes
Assessment against linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
嘿 @jbwfu - 我已经审查了你的更改,它们看起来很棒!
这是我在审查期间查看的内容
- 🟢 一般问题:一切看起来都很好
- 🟢 安全性:一切看起来都很好
- 🟢 测试:一切看起来都很好
- 🟢 复杂性:一切看起来都很好
- 🟢 文档:一切看起来都很好
帮助我变得更有用!请在每条评论上点击 👍 或 👎,我将使用反馈来改进你的审查。
Original comment in English
Hey @jbwfu - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: 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 #184
Summary by Sourcery
在安装脚本中添加对Linglong包管理器的支持,以增强LiteLoaderQQNT的脚本,使其更稳健地处理多种安装方法。
新功能:
增强功能:
Original summary in English
Summary by Sourcery
Add support for Linglong package manager in the installation script for LiteLoaderQQNT, enhancing the script to handle multiple installation methods more robustly.
New Features:
Enhancements: