jimorsm
jimorsm
> 请分离出具体的代码改动,不要把格式化和改动混合在一个 commit 中 还原了格式化的代码
> 大概清楚问题了。Github 的 releases api 具有分页的特性,要全部获取 releases 需要程序处理分页的逻辑。 > > 但是,如果应用该补丁,可能会造成服务器下载过多的资源。 > > 可以考虑将这个特性作为可选的,默认不开启,只下载第一页的 releases。 原始代码里就包含了下载的release数量限制,参考: - [github-release.json](https://github.com/tuna/tunasync-scripts/blob/master/github-release.json) - [github-release.py#L225](https://github.com/tuna/tunasync-scripts/blob/3cbfef6b4ecf6ae9dea97667479ff1c4ef403d6f/github-release.py#L225)
> 嗯,我看从功能上已经没什么问题了。但是,希望你能改进一下解析 `next_url` 的方法。你现在的做法看上去有点不是很健壮。这个位置其它有这种功能的库是怎么解析的? 参考GitHub API 文档 [创建分页方法的示例](https://docs.github.com/zh/enterprise-cloud@latest/rest/using-the-rest-api/using-pagination-in-the-rest-api?apiVersion=2022-11-28#example-creating-a-pagination-method) 改进了下代码,通过正则获取`next_url`
@shankerwangmiao @taoky 还有什么问题嘛?
@juewuy 由于我的调整涉及到ShellCrash.tar.gz,当前无法追踪压缩包内的修改,ShellCrash相关内容我如何提交补丁?
不能按照原作者的方式启动,需调整启动命令。可以参考我的仓库 ByZhouhang ***@***.***> 于2025年5月23日周五 05:41写道: > *ByZhouhang* left a comment (juewuy/ShellCrash#781) > > > 您好,我在 docker 内使用 shellcrash 的时候,按照作者提供的教程,经常遇到 resolv.conf 的复写错误,例如 sed: > can't move '/etc/resolv.confACFPEJ' to '/etc/resolv.conf': Resource busy,且最终无法成功使用代理...