Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: 更正CI运行时间cron表达 #77

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

WindowsMEMZ
Copy link
Contributor

按照11行注释说明,此ci应该在每天的2:51运行,而原cron表达式为每周五的2:31运行。

@WindowsMEMZ WindowsMEMZ requested a review from Cryolitia as a code owner June 2, 2024 16:14
@Zh40Le1ZOOB
Copy link
Member

按照11行注释说明,此ci应该在每天的2:51运行,而原cron表达式为每周五的2:31运行。

每天运行太频繁了,应该改注释才对

Cc @Cryolitia

@Cryolitia
Copy link
Member

把注释删了吧

@WindowsMEMZ
Copy link
Contributor Author

已修改

@Zh40Le1ZOOB
Copy link
Member

Zh40Le1ZOOB commented Jun 2, 2024

已修改

请变基你的分支,只保留修改注释的提交

同时,请修改提交说明,我建议删掉 scope 字段即可

@WindowsMEMZ
Copy link
Contributor Author

已修改

请变基你的分支,只保留修改注释的提交

同时,请修改提交说明,我建议删掉 scope 字段即可

@WindowsMEMZ WindowsMEMZ changed the title ci(flake-update-cron): 更正CI运行时间cron表达 ci: 更正CI运行时间cron表达 Jun 2, 2024
@Cryolitia
Copy link
Member

我解释一下,这段CI最早是我从NUR抄来的,用途也仅仅是在非flake下每天构建缓存而不是更新flake。

非flake下每次运行都会自动拉取最新的nixpkgs,仓库文件不记录rev;而flake下每次运行会拉取lock中指定的rev,需要显式的手动更新rev。

@Cryolitia Cryolitia merged commit 2e79bc0 into trans-archive:main Jun 2, 2024
3 checks passed
@WindowsMEMZ WindowsMEMZ deleted the patch-1 branch June 3, 2024 13:05
Zh40Le1ZOOB pushed a commit that referenced this pull request Sep 27, 2024
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.

3 participants