Skip to content

docs: 修正测试数、补安装说明、忽略 dsh-ops 残留 - #41

Merged
Nicholas023 merged 1 commit into
mainfrom
docs/readme-and-gitignore
Nov 4, 2026
Merged

Nicholas023 merged 1 commit into
mainfrom
docs/readme-and-gitignore

Conversation

@Nicholas023

Copy link
Copy Markdown
Owner

只改文档与 .gitignore

README

  • 测试数 331 → 491(重写后一直没更新)
  • 安装章节说清楚「怎么真的装进 DSH」:把包目录放进 profile 的 node_modules/,重启
  • 警告 npm 上没有本包:dsh-code-review 这个名字被别人占着(v0.1.2),pnpm add dsh-code-review 装到的是别的项目
  • 删掉一处重复的「不支持原地升级」提示

.gitignore

扫这个仓库时发现 .dsh-ops/panel/session-*.json/.ndjson(最大 5 MB)以未跟踪状态躺在工作区,
内含本机 Windows 用户名与绝对路径,而没有任何规则忽略它——距离被 git add -A 提进公开仓库只差一步。

新增:

.dsh-ops/
*/.dsh-ops/
.dsh-ops-update-*.lock

用 git check-ignore -v 验证过两条规则都命中。

无代码改动。

…esidue

README:
- 331 -> 491 tests (the number drifted since the rewrite).
- The install section now says what installing actually means (drop the
  package into the profile node_modules and restart) and warns that the npm
  name dsh-code-review belongs to someone else, so `pnpm add dsh-code-review`
  installs a different project.
- Drops a duplicated upgrade note; the install section owns it now.

.gitignore:
- Ignores .dsh-ops/ and its update lock files. A scan of this repo found that
  .dsh-ops/panel/session-*.json/.ndjson (up to 5 MB) sat untracked in the
  working tree, carrying the local Windows user name and absolute paths, and
  nothing ignored them - one `git add -A` from a public commit.
@Nicholas023
Nicholas023 merged commit fe95c2a into main Nov 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant