Releases with non-version tags are named d-<tag> instead of dev-<tag>
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
从 src/Repository/Release.php 中的 simplifyReleaseName() 开始。该函数会截掉以连字符分隔的第一部分的最后两个字符,因此会把 dev-nightly 变成 d-nightly。阅读 src/Repository/GitHub/GitHubRelease.php 中的 GitHubRelease::getTagName(),了解已经正常工作的 dev- 回退逻辑。删除 tests/Unit/Repository/GitHub/GitHubReleaseTest.php 中 fromApiResponseNamesUnparsableReleaseAsDevBranch 上的 #[Skip],然后运行 vendor/bin/testo --filter=fromApiResponseNamesUnparsableReleaseAsDevBranch。当该测试通过,且数字版本仍只去掉结尾的 .0 时,即完成。Issue #49 可能会替换这个查找逻辑,因此请先检查它。
由索引模型根据 Issue 内容生成。
描述
Description
A GitHub release whose tag and name are not valid versions is named d-<tag> instead of dev-<tag>. For a release with tag_name: nightly and name: Nightly, getName() returns d-nightly. The versions command shows this name, and satisfies() checks constraints against it.
GitHubRelease::getTagName() correctly falls back to dev-nightly. Release::simplifyReleaseName() then normalizes the name, splits it on - and always cuts the last two characters of the first part. That step is meant to strip the .0 that VersionParser::normalize() appends to numeric versions (2024.1.0.0 becomes 2024.1.0), but for dev-nightly it turns dev into d.
Steps to reproduce
- Remove
#[Skip]fromGitHubReleaseTest::fromApiResponseNamesUnparsableReleaseAsDevBranch. - Run
vendor/bin/testo --filter=fromApiResponseNamesUnparsableReleaseAsDevBranch.
Additional context
The release lookup may be replaced by DLoad in #49, in which case this code goes away.
- 主要语言
- PHP
- 星标
- 30
- 派生
- 15
- 平均合并
- 9 小时 37 分钟
- 30 天内合并 PR
- 16
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
roadrunner-php/cli 的其他 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 72/100
roadrunner-php/cli#67 ·
维护者通常 1 天内回复
查看 roadrunner-php/cli 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复
-
Do not throw `WordPress.Security.EscapeOutput.OutputNotEscaped` on `apply_shortcodes()`可能已有人在做 @tbradsha 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 62/100
landsman/adminer-desktop#120 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
components-web-app/docs#227 ·
-
难度 1/5 1 小时以内 新手友好度 85/100