pnpm run lint does not lint the e2e tests
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 78/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- eslint, typescript
调研方向
从 package.json 中的 lint 脚本和 e2e/cli-publish.test.ts:109 开始；运行报告中的 ESLint 命令，确认现有错误。检查 libs/ 中的包如何组织其 e2e/ 文件夹，然后扩大 lint 覆盖范围并修复报告的非空断言。完成标准是 pnpm run lint 检查根目录和库中的端到端测试并通过。
由索引模型根据 Issue 内容生成。
描述
Which package is this bug report for?
@nanoforge-dev/cli
Which command is this bug report for?
other
Issue description
pnpm run lint runs eslint on src only (package.json#L47: prettier --check . && eslint --format=pretty src), so e2e/ is never linted. It already holds lint errors:
pnpm exec eslint e2e/cli-publish.test.ts
109:12 error Forbidden non-null assertion @typescript-eslint/no-non-null-assertion
(cli-publish.test.ts#L109, reported twice on that line.) Found while reviewing the Windows test fixes (#274).
Expected: the lint script covers e2e/ (and the libs' e2e/ folders), and the existing errors are fixed.
Code sample
// e2e/cli-publish.test.ts:109 (main)
expect(put!.files!.sort()).toEqual(["index.ts", "nanoforge.manifest.json", "src/player.ts"]);
Versions
- @nanoforge-dev/cli
main(e95b881) - Node.js 26, Linux
Issue priority
Low (slightly annoying)
I have tested this issue on a development release
e95b881
- 主要语言
- TypeScript
- 星标
- 2
- 派生
- 2
- 平均合并
- 2 天 18 小时
- 30 天内合并 PR
- 17
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
NanoForge-dev/CLI 的其他 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 88/100
NanoForge-dev/CLI#294 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 76/100
NanoForge-dev/CLI#292 ·
维护者通常 1 天内回复
-
command:dev enhancement good first issue refactor
难度 2/5 1-3 小时 新手友好度 78/100
NanoForge-dev/CLI#281 ·
维护者通常 1 天内回复
-
bug command:build good first issue
难度 2/5 1-3 小时 新手友好度 88/100
NanoForge-dev/CLI#276 ·
维护者通常 1 天内回复
-
documentation good first issue
难度 1/5 1 小时以内 新手友好度 74/100
NanoForge-dev/CLI#228 ·
维护者通常 1 天内回复
查看 NanoForge-dev/CLI 的全部 Issue
相似的 Issue
-
Mend: dependency security vulnerability
难度 1/5 1-3 小时 新手友好度 65/100
fdm-monster/fdm-monster#5563 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
难度 2/5 1-3 小时 新手友好度 79/100
维护者通常 1 天内回复
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
难度 2/5 1-3 小时 新手友好度 65/100
growilabs/growi#12246 · 3 条评论 ·
维护者通常 1 天内回复
-
🐛 The Rulekeeper - An index run that gives up on the lock ends Succeeded可能已有人在做 @github-actions 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100
SSWConsulting/SSW.Rules#2809 · 1 条评论 ·
维护者通常 1 天内回复