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.

Sign up for GitHub

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

Jump to bottom

feat(route): 色花堂添加版块 #8696

Merged
merged 5 commits into from
Feb 25, 2022
Merged

feat(route): 色花堂添加版块 #8696

merged 5 commits into from
Feb 25, 2022

Conversation

hideOnBushhh
Copy link

该 PR 相关 Issue / Involved issue

Close 无

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

NOROUTE

新RSS检查列表 / New RSS Script Checklist

  • New Route
  • Documentation
    • CN
    • EN
  • 全文获取 fulltext
    • Use Cache
  • 反爬/频率限制 anti-bot or rate limit?
    • 如果有, 是否有对应的措施? If yes, do your code reflect this sign?
  • 日期和时间 date and time
    • 可以解析 Parsed
    • 时区调整 Correct TimeZone
  • 添加了新的包 New package added
  • Puppeteer

说明 / Note

色花堂添加综合讨论区并修改doc

@github-actions github-actions bot added the Auto: Route Test Skipped PR involves no routes label Dec 8, 2021
TonyRL
TonyRL requested changes Dec 8, 2021
View reviewed changes
lib/routes/dsndsht23/index.js Outdated Show resolved Hide resolved
@vercel
Copy link

vercel bot commented Dec 9, 2021 •
edited
Loading

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/diy/rsshub/6tqfnyWhvaqKMeqQgiZSbaiUUgzb
✅ Preview: https://rsshub-git-fork-hideonbushhh-master-diy.vercel.app

TonyRL
TonyRL requested changes Jan 12, 2022
View reviewed changes
Dockerfile Outdated
Comment on lines 22 to 24
unset PUPPETEER_SKIP_CHROMIUM_DOWNLOAD && npm ci ;\
unset PUPPETEER_SKIP_CHROMIUM_DOWNLOAD && npm i ;\
else \
export PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true && npm ci ;\
export PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true && npm i ;\
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please pull the latest changes from master branch.
Building issue has been fixed in b36b326

Dockerfile Outdated
ARG USE_CHINA_NPM_REGISTRY=0;
ARG USE_CHINA_NPM_REGISTRY=1;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do not change this.

@vercel
Copy link

vercel bot commented Feb 25, 2022

Deployment failed with the following error:

Your Personal Account exceeded our fair use limits and has been blocked.

Learn More: http://vercel.link/fair-use

@TonyRL TonyRL merged commit 6cfa103 into DIYgod:master Feb 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TonyRL TonyRL TonyRL requested changes

Assignees
No one assigned
Labels
Auto: Route Test Skipped PR involves no routes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hideOnBushhh @TonyRL