marvin8/dujiangyan
Watch
1
0
Fork
You've already forked dujiangyan
 0
Code Issues Pull requests Projects Releases 43 Wiki Activity Actions

Migrate CI to Forgejo Actions and add Release-Notes workflow #70

Merged
coding-agent-marvin8 merged 0 commits from refs/pull/70/head into main 2026-07-12 08:46:40 +00:00
Conversation 3 Commits - Files changed -
coding-agent-marvin8 commented 2026-07-12 07:00:35 +00:00 (Migrated from codeberg.org)
Copy link

Migrates all Woodpecker CI pipelines to Forgejo Actions, adds per-package Release-Notes.md with taibai thread integration, drops git-cliff, and adds ai-declaration.md.

Changes

  • Add .forgejo/workflows/fenliu-ci.yml and zhongli-ci.yml with path-filtered PR triggers and tag-triggered publish/deploy-docs
  • Add .forgejo/workflows/ai-review.yml (WuMing on all PRs) and weekly-checks.yml (cron 02:27 AEST Saturday on codeberg-small-lazy)
  • Add per-package Release-Notes.md and Release-Notes.md.template; bumpversion configured to update version line on release
  • Refactor both justfiles: drop git-cliff, generate CHANGELOG from Release-Notes bullets + commit links with Codeberg URLs
  • Add fediverse-validate and fediverse-post just recipes (default profile: announce)
  • Remove .woodpecker/ pipelines, both cliff.toml files, and git-cliff dependency
  • Add ai-declaration.md at repo root
  • Update CLAUDE.md files (root, fenliu, zhongli) and RELEASE-COMMANDS.md

Secrets to add in Codeberg repo settings

The new workflows reference two new secret names (underscores, Forgejo Actions convention):

  • pypi_fenliu_token — was pypi-fenliu-token in Woodpecker
  • pypi_zhongli_token — was pypi-zhongli-token in Woodpecker
  • wuming_deepseek_key — same name as before (already exists)

Closes #69

Migrates all Woodpecker CI pipelines to Forgejo Actions, adds per-package Release-Notes.md with taibai thread integration, drops git-cliff, and adds ai-declaration.md. ## Changes - Add `.forgejo/workflows/fenliu-ci.yml` and `zhongli-ci.yml` with path-filtered PR triggers and tag-triggered publish/deploy-docs - Add `.forgejo/workflows/ai-review.yml` (WuMing on all PRs) and `weekly-checks.yml` (cron 02:27 AEST Saturday on `codeberg-small-lazy`) - Add per-package `Release-Notes.md` and `Release-Notes.md.template`; bumpversion configured to update version line on release - Refactor both justfiles: drop git-cliff, generate CHANGELOG from Release-Notes bullets + commit links with Codeberg URLs - Add `fediverse-validate` and `fediverse-post` just recipes (default profile: `announce`) - Remove `.woodpecker/` pipelines, both `cliff.toml` files, and `git-cliff` dependency - Add `ai-declaration.md` at repo root - Update `CLAUDE.md` files (root, fenliu, zhongli) and `RELEASE-COMMANDS.md` ## Secrets to add in Codeberg repo settings The new workflows reference two new secret names (underscores, Forgejo Actions convention): - `pypi_fenliu_token` — was `pypi-fenliu-token` in Woodpecker - `pypi_zhongli_token` — was `pypi-zhongli-token` in Woodpecker - `wuming_deepseek_key` — same name as before (already exists) Closes #69
coding-agent-marvin8 commented 2026-07-12 07:43:47 +00:00 (Migrated from codeberg.org)
Copy link

line 36

Incorrect mypy ignore comment: "# ty: ignore[missing-argument]" uses "ty" instead of "type"…

🔴 This project uses ty (Astral's type checker), not mypy. ty has its own ignore syntax: # ty: ignore[rule], which is distinct from mypy's # type: ignore[rule]. The # ty: ignore[missing-argument] comment is correct and intentional. The pre-existing # type: ignore[call-arg] on the same line already handles mypy/editors.

[ line 36](https://codeberg.org/marvinsmastodontools/dujiangyan/pulls/70#issuecomment-19055684) > Incorrect mypy ignore comment: "# ty: ignore[missing-argument]" uses "ty" instead of "type"… 🔴 This project uses [ty](https://github.com/astral-sh/ty) (Astral's type checker), not mypy. ty has its own ignore syntax: `# ty: ignore[rule]`, which is distinct from mypy's `# type: ignore[rule]`. The `# ty: ignore[missing-argument]` comment is correct and intentional. The pre-existing `# type: ignore[call-arg]` on the same line already handles mypy/editors.
coding-agent-marvin8 commented 2026-07-12 07:44:03 +00:00 (Migrated from codeberg.org)
Copy link

packages/fenliu/tests/test_schemas.py line 265

Incorrect mypy ignore comment: "# ty: ignore[missing-argument]" should be "# type: ignore[missing-argument]"…

🔴 Same as above — this project uses ty (Astral type checker), not mypy. ty uses its own ignore syntax (# ty: ignore[rule]), distinct from mypy (# type: ignore[rule]). The comment is correct; no change needed.

[packages/fenliu/tests/test_schemas.py line 265](https://codeberg.org/marvinsmastodontools/dujiangyan/pulls/70#issuecomment-19055687) > Incorrect mypy ignore comment: "# ty: ignore[missing-argument]" should be "# type: ignore[missing-argument]"… 🔴 Same as above — this project uses [ty](https://github.com/astral-sh/ty) (Astral type checker), not mypy. ty uses its own ignore syntax (# ty: ignore[rule]), distinct from mypy (# type: ignore[rule]). The comment is correct; no change needed.
forgejo-actions commented 2026-07-12 08:28:53 +00:00 (Migrated from codeberg.org)
Copy link

This PR is too large to review automatically (4651 lines; limit is 3000). Please consider splitting it into smaller PRs.

This PR is too large to review automatically (4651 lines; limit is 3000). Please consider splitting it into smaller PRs.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
marvin8/dujiangyan!70
No description provided.