Skip to content

Sync 2021-8-24#7

Merged
youngH1001 merged 67 commits intoyoungH1001:masterfrom
DIYgod:master
Aug 30, 2021
Merged

Sync 2021-8-24#7
youngH1001 merged 67 commits intoyoungH1001:masterfrom
DIYgod:master

Conversation

@youngH1001
Copy link
Copy Markdown
Owner

该 PR 相关 Issue / Involved issue

Close #

完整路由地址 / Example for the proposed route(s)

新RSS检查列表 / New RSS Script Checklist

  • 这个PR中包含了新的路由吗? Does this PR add new route?
    • 如果有, 请完成检查列表. If yes, please finish the check list
    • 如果你的PR符合下方某个事项, 也请注明. If any of the checklist item meets your PR, please fill it out.
    • <- 这样打勾
  • 是否提供了文档? Documentation provided?
    • 是否提供了英文文档? EN Documentation provided?
  • 是否支持全文获取? Is this RSS Script support fulltext?
    • 如果全文获取中需要访问文章链接, 是否使用了缓存? If fulltext requires to fetch detail pages, is cache used in the process?
    • 缓存说明 | How to use cache
  • 目标是否有明显的反爬/频率限制? Is there any sign of anti-bot or rate limit?
    • 如果有, 是否有对应的措施? (延长缓存时间, 写文档说明, etc.) If yes, do your code reflect this sign? (e.g. write documentations, use long cache time)
  • 目标是否有提供日期? Is there a date in the source?
    • 如果有,包是否正确解析? If there is, can this script provide this info?
    • 如果有提供解析能力,时区是否正确调整? Is the timezone correctly provided?
    • 如果有提供日期,但是没有提供解析,请说明原因 If there is a date but this script does not parse, please provide your reason.
  • 是否引入的新的包? Any new package introduced?
    • 如果有, 请说明原因. If yes, please state your reason
  • 是否使用了Puppeteer? Make use of Puppeteer?
    • 如果有, 请说明原因. If yes, please state your reason

说明 / Note

zytomorrow and others added 30 commits August 11, 2021 22:59
Before this fix, timeout messages are not quite useful

> error: Request undefined fail, retry attempt #1: TimeoutError: Timeout awaiting 'request' for 5000ms
actions-user and others added 26 commits August 12, 2021 09:28
*some of them shown cancelled instead of failed. Need to address this issue later
heroku补充描述
heroku additional description
vercel 补充描述
vercel additional description
heroku补充描述
heroku additional description
vercel 补充描述
vercel additional description
* fix(dockerfile): copy necessary file when building

*when even bother deleting file...

* no console
* chore(test): test running docker after build

* add test in the same step

* add count times

* workflow typo

* add name when output

* if then

* test docker

* use id

* no tag

* what is docker id

* load single platform

* do not test multi-arch for now

* no load
* style: prefer object shorthand syntax
* refactor: prefer Array#map over Array#forEach
* style: prefer arrow callback
* chore(eslint): update rules
* style: auto fix by eslint
@youngH1001 youngH1001 marked this pull request as ready for review August 30, 2021 15:58
@youngH1001 youngH1001 merged commit e77415d into youngH1001:master Aug 30, 2021
youngH1001 added a commit that referenced this pull request Sep 4, 2021
@youngH1001 youngH1001 mentioned this pull request Sep 4, 2021
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.