-
-
Notifications
You must be signed in to change notification settings - Fork 499
opt.: new website #1138
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
opt.: new website #1138
Changes from all commits
b77377e
3c22a46
7bf3894
acada65
25ec515
b553f47
d7bd5db
73e5688
dd059d0
18fc824
4a0001e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -30,9 +30,10 @@ Especially thanks to <a href="https://github.com/TerminalStudio/dartssh2">dartss | |
|
|
||
| |Platform| From| | ||
| |--|--| | ||
| | iOS / macOS | [AppStore](https://apps.apple.com/app/id1586449703) | | ||
| | Android | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) / [F-Droid](https://f-droid.org/packages/tech.lolli.toolbox) / [OpenAPK](https://www.openapk.net/serverbox/tech.lolli.toolbox/) | | ||
| | Linux / Windows | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) | | ||
| | iOS | [AppStore](https://apps.apple.com/app/id1586449703) | | ||
| | macOS | [AppStore](https://apps.apple.com/app/id1586449703) / brew install --cask server-box | | ||
| | Android | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) / [F-Droid](https://f-droid.org/packages/tech.lolli.toolbox) / [OpenAPK](https://www.openapk.net/serverbox/tech.lolli.toolbox/) | | ||
| | Linux / Windows | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) | | ||
|
Comment on lines
+33
to
+36
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix markdownlint MD055 table pipe style on installation rows. Line 33 and Line 34 currently violate the configured table pipe style; this can cause lint noise/failures. 🔧 Proposed markdownlint-compliant table format-|Platform| From|
-|--|--|
-| iOS | [AppStore](https://apps.apple.com/app/id1586449703) |
-| macOS | [AppStore](https://apps.apple.com/app/id1586449703) / brew install --cask server-box |
-| Android | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) / [F-Droid](https://f-droid.org/packages/tech.lolli.toolbox) / [OpenAPK](https://www.openapk.net/serverbox/tech.lolli.toolbox/) |
-| Linux / Windows | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) |
+Platform | From
+-- | --
+iOS | [AppStore](https://apps.apple.com/app/id1586449703)
+macOS | [AppStore](https://apps.apple.com/app/id1586449703) / brew install --cask server-box
+Android | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) / [F-Droid](https://f-droid.org/packages/tech.lolli.toolbox) / [OpenAPK](https://www.openapk.net/serverbox/tech.lolli.toolbox/)
+Linux / Windows | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid)🧰 Tools🪛 markdownlint-cli2 (0.22.1)[warning] 33-33: Table pipe style (MD055, table-pipe-style) [warning] 33-33: Table pipe style (MD055, table-pipe-style) [warning] 34-34: Table pipe style (MD055, table-pipe-style) [warning] 34-34: Table pipe style (MD055, table-pipe-style) 🤖 Prompt for AI Agents |
||
|
|
||
| Please only download pkgs from the source that **you trust**! | ||
|
|
||
|
|
@@ -73,17 +74,6 @@ If I forgot to add your name to the contributors list, please add a comment in t | |
| 2. Clone this repo, run `flutter run` to start the app. | ||
| 3. Run `dart run fl_build -p PLATFORM` to build the app. | ||
|
|
||
| ### Release macOS notarized DMG | ||
|
|
||
| 1. Copy `.env.release.example` to `.env.release`. | ||
| 2. Fill in `APPLE_TEAM_ID` and `APPLE_NOTARY_KEYCHAIN_PROFILE`. | ||
| 3. Make sure the `Developer ID Application` certificate is already installed in Keychain. | ||
| 4. Make sure notarization credentials are already stored via `xcrun notarytool store-credentials`. | ||
| 5. Install the provisioning profile used for DMG packaging. The script defaults to `ServerBox DMG Profile`, and you can override it with `APP_PROFILE_NAME`. | ||
| 6. Run `bash scripts/release/release-macos-dmg.sh`. | ||
|
|
||
| This flow does not modify the default Xcode Release signing config. It injects a temporary `xcconfig` only for archive/export, builds a signed `.app`, packages a DMG, submits it to notarization, staples the result, and optionally uploads the DMG to the GitHub Release for `v<version>`. | ||
|
|
||
| ### Translation | ||
|
|
||
| - [Guide](https://blog.lpkt.cn/posts/faq/) can be found in my blog. | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -30,8 +30,9 @@ | |||||||||
|
|
||||||||||
| 平台|下载 | ||||||||||
| --|-- | ||||||||||
| iOS / macOS | [AppStore](https://apps.apple.com/app/id1586449703) | ||||||||||
| Android | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) / [F-Droid](https://f-droid.org/packages/tech.lolli.toolbox) / [OpenAPK](https://www.openapk.net/serverbox/tech.lolli.toolbox/) | ||||||||||
| | iOS | [AppStore](https://apps.apple.com/app/id1586449703) | | ||||||||||
| | macOS | [AppStore](https://apps.apple.com/app/id1586449703) / brew install --cask server-box | | ||||||||||
|
Comment on lines
+33
to
+34
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix table row pipe style to satisfy markdownlint MD055. Line 33 and Line 34 use leading/trailing pipes, but this table is otherwise in compact style. This will keep lint warnings active. 💡 Suggested diff-| iOS | [AppStore](https://apps.apple.com/app/id1586449703) |
-| macOS | [AppStore](https://apps.apple.com/app/id1586449703) / brew install --cask server-box |
+iOS | [AppStore](https://apps.apple.com/app/id1586449703)
+macOS | [AppStore](https://apps.apple.com/app/id1586449703) / brew install --cask server-box🧰 Tools🪛 markdownlint-cli2 (0.22.1)[warning] 33-33: Table pipe style (MD055, table-pipe-style) [warning] 33-33: Table pipe style (MD055, table-pipe-style) [warning] 34-34: Table pipe style (MD055, table-pipe-style) [warning] 34-34: Table pipe style (MD055, table-pipe-style) 🤖 Prompt for AI Agents
Comment on lines
+33
to
+34
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🟡 README_zh.md table rows have inconsistent pipe formatting causing broken table The installation table in README_zh.md mixes two markdown table row styles. Lines 33–34 (iOS, macOS) use
Suggested change
Was this helpful? React with 👍 or 👎 to provide feedback. |
||||||||||
| Android | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) / [F-Droid](https://f-droid.org/packages/tech.lolli.toolbox) / [OpenAPK](https://www.openapk.net/serverbox/tech.lolli.toolbox/) | ||||||||||
| Linux / Windows | [GitHub](https://github.com/lollipopkit/flutter_server_box/releases) / [CDN](https://cdn.lpkt.cn/serverbox/pkg/?sort=time&order=desc&layout=grid) | ||||||||||
|
|
||||||||||
| 请从 **信任** 的来源下载! | ||||||||||
|
|
@@ -75,17 +76,6 @@ Linux / Windows | [GitHub](https://github.com/lollipopkit/flutter_server_box/rel | |||||||||
| 2. 克隆这个仓库, 运行 `flutter run` 启动应用 | ||||||||||
| 3. 运行 `dart run fl_build -p PLATFORM` 构建应用 | ||||||||||
|
|
||||||||||
| ### 发布 macOS 公证 DMG | ||||||||||
|
|
||||||||||
| 1. 复制 `.env.release.example` 为 `.env.release` | ||||||||||
| 2. 填入 `APPLE_TEAM_ID` 和 `APPLE_NOTARY_KEYCHAIN_PROFILE` | ||||||||||
| 3. 确保 `Developer ID Application` 证书已经安装到 Keychain | ||||||||||
| 4. 确保已经通过 `xcrun notarytool store-credentials` 存好了公证凭据 | ||||||||||
| 5. 安装用于 DMG 打包的 provisioning profile。脚本默认使用 `ServerBox DMG Profile`,也可以通过 `APP_PROFILE_NAME` 覆盖 | ||||||||||
| 6. 运行 `bash scripts/release/release-macos-dmg.sh` | ||||||||||
|
|
||||||||||
| 这套流程不会修改工程默认的 Xcode Release 签名配置。脚本只会在归档和导出时注入临时 `xcconfig`,生成已签名 `.app`、打包 DMG、提交公证、回填 stapler,并可选把 DMG 上传到 `v<version>` 对应的 GitHub Release。 | ||||||||||
|
|
||||||||||
| ### 翻译 | ||||||||||
|
|
||||||||||
| [指南](https://blog.lpkt.cn/faq/) 可在我的博客中找到。 | ||||||||||
|
|
||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,17 +18,7 @@ flutter test --coverage | |
|
|
||
| ## Teststruktur | ||
|
|
||
| Tests befinden sich im Verzeichnis `test/` und spiegeln die Struktur von `lib/` wider: | ||
|
|
||
| ``` | ||
| test/ | ||
| ├── data/ | ||
| │ ├── model/ | ||
| │ └── provider/ | ||
| ├── view/ | ||
| │ └── widget/ | ||
| └── test_helpers.dart | ||
| ``` | ||
| Tests befinden sich im Verzeichnis `test/`. Die aktuelle Suite ist überwiegend flach und nach Parser-, Modell- und Utility-Verhalten gruppiert, zum Beispiel `cpu_test.dart`, `container_test.dart` und `ssh_config_test.dart`. | ||
|
|
||
| ## Unit-Tests | ||
|
|
||
|
|
@@ -71,26 +61,13 @@ test('serverStatusProvider gibt Status zurück', () async { | |
| }); | ||
| ``` | ||
|
|
||
| ## Mocking | ||
|
|
||
| Mocks für externe Abhängigkeiten verwenden: | ||
|
|
||
| ```dart | ||
| class MockSshService extends Mock implements SshService {} | ||
| ## Externe Abhängigkeiten | ||
|
|
||
| test('verbindet zum Server', () async { | ||
| final mockSsh = MockSshService(); | ||
| when(mockSsh.connect(any)).thenAnswer((_) async => true); | ||
|
|
||
| // Test mit Mock | ||
| }); | ||
| ``` | ||
| Vermeiden Sie Tests, die von echten SSH-Servern abhängen. Parser-, Modell- und Command-Builder-Tests sollten deterministisch bleiben; fügen Sie gezielte Fakes oder Fixtures hinzu, wenn eine Funktion eine Service-Grenze einführt. | ||
|
|
||
| ## Integrationstests | ||
|
|
||
| Komplette Benutzerabläufe testen (in `integration_test/`): | ||
|
|
||
| ```dart | ||
| Im aktuellen Repository gibt es keine `integration_test/`-Suite. Fügen Sie Integrationstests nur hinzu, wenn eine Funktion End-to-End-Geräte- oder App-Flow-Abdeckung benötigt.dart | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🟡 Missing code fence opener in localized testing docs renders Dart code as plain paragraph text In the German (and identically in es, fr, ja, zh) testing docs, the Same bug in other locale files
The English version ( Prompt for agentsWas this helpful? React with 👍 or 👎 to provide feedback. |
||
| testWidgets('Server hinzufügen Ablauf', (tester) async { | ||
| await tester.pumpWidget(MyApp()); | ||
|
|
||
|
|
||
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.
🚩 Makefile gen-build-clean target still uses invalid --clean flag despite docs update
The PR updates documentation in multiple languages (e.g.,
docs/src/content/docs/development/codegen.md:24-28) to recommenddart run build_runner cleanfollowed bydart run build_runner build --delete-conflicting-outputsas two separate commands. However, the Makefilegen-build-cleantarget atMakefile:94-95still uses the old single-command formdart run build_runner build --delete-conflicting-outputs --clean. The--cleanflag is not a valid option forbuild_runner build; the correct approach isbuild_runner cleanas a separate subcommand. This is a pre-existing issue, but the PR's doc changes explicitly corrected the guidance without updating the Makefile target to match.(Refers to lines 94-95)
Was this helpful? React with 👍 or 👎 to provide feedback.