Skip to content

refactor: remove agfs port configuration#483

Merged
MaojiaSheng merged 1 commit intomainfrom
remove-agfs-port
Mar 9, 2026
Merged

refactor: remove agfs port configuration#483
MaojiaSheng merged 1 commit intomainfrom
remove-agfs-port

Conversation

@qin-ctx
Copy link
Collaborator

@qin-ctx qin-ctx commented Mar 9, 2026

Description

移除 AGFS 端口配置,因为 AGFS 不再需要显式配置端口。

Type of Change

  • Refactoring (no functional changes)
  • Documentation update

Changes Made

  • 从 INSTALL.md 和 INSTALL-ZH.md 中移除 agfs port 配置示例
  • 从 install.ps1 中移除 AGFS 端口相关配置
  • 从 install.sh 中移除 AGFS 端口相关配置
  • 从 setup-helper/cli.js 中移除 AGFS 端口相关配置

Checklist

  • My code follows the project's coding style
  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

PR Code Suggestions ✨

No code suggestions found for the PR.

@MaojiaSheng MaojiaSheng merged commit 4fd6ff1 into main Mar 9, 2026
20 of 22 checks passed
@MaojiaSheng MaojiaSheng deleted the remove-agfs-port branch March 9, 2026 08:47
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenViking project Mar 9, 2026
Soda-Wong added a commit that referenced this pull request Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants