feat: DSH source rebased after crate flatten; drop .cursor/.github - #115
Merged
Conversation
These were editor/CI overlays, not the product. Release and PR checks are no longer defined in-tree.
Replay reviewed PR #112 onto current main: rust/src → src, drop the deleted TypeScript oracle edits, add dsh to eval/types, keep zstd decode and native tests.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This was referenced Aug 17, 2026
pull Bot
pushed a commit
to CrazyForks/sherlog
that referenced
this pull request
Aug 17, 2026
These were removed with .cursor/.github in catoncat#115. Bring back ci.yml, release.yml, and the PR template so tag-triggered native archives work again.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
为什么不是直接合 #112
#112 基于旧
main（53029a9），而且改的是已经不存在的路径：rust/src/**— chore: drop TypeScript CLI and flatten Rust crate to repo root #114 已摊到仓库根src/src/**/*.ts— TypeScript 对照 CLI 已删除rust/Cargo.toml— 已并入根Cargo.tomlGitHub 上 #112 状态是 CONFLICTING。不能在 UI 里按 Merge。
这个 PR 做了什么
.cursor/和.github/（含 CI / release workflow）main：src/sources/dsh.rs+ catalog/identity/config/zstdeval/types.ts增加dsh（不再改已删除的 TS CLI）src/*.tsoracle stub（没有对照机可 stub 了）行为应与 #112 review 后的 native 部分相同。acceptance 仍不造 zstd fixture；DSH 覆盖在 Rust e2e。
怎么合
合这个 PR，然后关掉 #112（或标 superseded）。不要把 #112 再 rebase 回
feat/dsh-source除非要保留那条评论线程。注意：去掉
.github/workflows之后 GitHub Actions 不会再跑。