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

fix(#49): github api rate limit in actions #53

Merged
merged 1 commit into from
Jan 13, 2024
Merged

fix(#49): github api rate limit in actions #53

merged 1 commit into from
Jan 13, 2024

Conversation

Oscaner
Copy link
Member

@Oscaner Oscaner commented Jan 12, 2024

No description provided.

@Oscaner
Copy link
Member Author

Oscaner commented Jan 12, 2024

image

@ivaquero 我们创建一个 Brewforge-CI 的组织,专门来做 bump fork 的工作吧?目前被挡在这个位置了。

@ivaquero
Copy link
Contributor

可以啊,你可以来创建么,你比我们在行

@Oscaner
Copy link
Member Author

Oscaner commented Jan 12, 2024

@ivaquero @Mrered 我创建了 Brewforge-Bot,邀请你们了。

@Mrered
Copy link
Member

Mrered commented Jan 12, 2024

@ivaquero @Mrered 我创建了 Brewforge-Bot,邀请你们了。

Me in

Copy link
Member

@Mrered Mrered left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不会写但是能大概看懂

@ivaquero
Copy link
Contributor

加了

@ivaquero
Copy link
Contributor

这个需要合并了@我,目前我还看不大明白

@Oscaner
Copy link
Member Author

Oscaner commented Jan 12, 2024

好的。token 权限还有问题,还在调。

@Oscaner Oscaner force-pushed the fix-api-rate branch 14 times, most recently from 7a3fd90 to b69ea62 Compare January 12, 2024 16:09
@ivaquero
Copy link
Contributor

强大

@Oscaner Oscaner force-pushed the fix-api-rate branch 2 times, most recently from ad7ef12 to 83f553d Compare January 12, 2024 16:19
@Oscaner Oscaner force-pushed the fix-api-rate branch 3 times, most recently from 060d1f2 to 709836f Compare January 13, 2024 07:55
@ivaquero
Copy link
Contributor

这次算是OK了,对吧 @Oscaner?上面两个提交我没看CI结果

@ivaquero
Copy link
Contributor

目前貌似还是权限的问题,Casks 不过没关系,Formulae 过了就可以

@Oscaner
Copy link
Member Author

Oscaner commented Jan 13, 2024

就是 Formula 过不了。组织的 PAT 权限有缺陷。个人的 PAT 卡在 commit push。
目前给的权限是 repo, workflow 和 gist。我看 homebrew 的源码里也就只需要这三个 scope 的权限。
我拿同样的 token 在本地执行都是 OK 的,这就很奇怪。

@Oscaner Oscaner force-pushed the fix-api-rate branch 2 times, most recently from ce05d85 to d3e2b67 Compare January 13, 2024 08:44
@ivaquero
Copy link
Contributor

我之前尝试过PAT,和你的结果一样

@Oscaner Oscaner force-pushed the fix-api-rate branch 12 times, most recently from b6077b6 to 279d473 Compare January 13, 2024 12:05
@Oscaner
Copy link
Member Author

Oscaner commented Jan 13, 2024

@ivaquero @Mrered
目前创建 PR 已经没问题了。先合并一版进 main 分支,后续有问题再修。
我创建了一个 task,对 action 再做改进。如果有什么想法也可以在 comment 里补充。
Action 改进 #59

@Oscaner Oscaner merged commit 1f63559 into main Jan 13, 2024
@Oscaner Oscaner deleted the fix-api-rate branch January 13, 2024 12:26
@Mrered
Copy link
Member

Mrered commented Jan 15, 2024

之前手动触发 Actions 时见到过 GitHub API 调用超频警告

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