Skip to content

Conversation

johan456789
Copy link

No description provided.

@follow-reviewer-bot
Copy link

Thank you for your contribution. We will review it promptly.

Copy link

netlify bot commented Aug 15, 2025

Deploy Preview for rsshub-docs-next ready!

Name Link
🔨 Latest commit 55c5d45
🔍 Latest deploy log https://app.netlify.com/projects/rsshub-docs-next/deploys/68a614d36b4cbd0008dfa220
😎 Deploy Preview https://deploy-preview-122--rsshub-docs-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@DIYgod
Copy link
Member

DIYgod commented Aug 17, 2025

The new address is a suspicious maintainer's address, and I prefer to remove this link rather than switch to an unsafe link. cnpm/cnpmcore#777

@johan456789
Copy link
Author

johan456789 commented Aug 17, 2025

The new address is a suspicious maintainer's address, and I prefer to remove this link rather than switch to an unsafe link. cnpm/cnpmcore#777

Interesting, I originally thought aui changed their username. DIdn't know about this.

How about linking to archived links? Still https://aui.github.io/art-template/ it's just snapshots from Wayback Machine. It's better to provide documentation this way then letting users googling their way to the goofychris one.

@TonyRL
Copy link
Contributor

TonyRL commented Aug 18, 2025

There're few notes on this matter in https://github.com/DIYgod/RSSHub/blob/60ba70d139128ad2c951211ddb1891eaac14d3c7/.github/dependabot.yml#L19.
Linking to the syntax maybe better than the docs landing page since web archive takes some time to load.
I may migrate to another template engine when I have some free time.

@johan456789
Copy link
Author

Linking to the syntax maybe better than the docs landing page since web archive takes some time to load.

That's a good point. fixed.

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