Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove obsolete redirect docs from /doc/{env,json,cli}.md#596

Merged
NewFuture merged 5 commits intomasterfrom
copilot/remove-old-redirect-docs
Nov 15, 2025
Merged

Remove obsolete redirect docs from /doc/{env,json,cli}.md#596
NewFuture merged 5 commits intomasterfrom
copilot/remove-old-redirect-docs

Conversation

Copy link
Contributor

Copilot AI commented Nov 15, 2025

Removed old manual redirect stub files from the /doc/ directory that were pointing to their new locations in /doc/config/.

Changes

  • Deleted redirect stubs: Removed doc/{env,json,cli}.{md,en.md} (6 files total)
    • doc/env.md → redirected to config/env.md
    • doc/env.en.md → redirected to config/env.en.md
    • doc/json.md → redirected to config/json.md
    • doc/json.en.md → redirected to config/json.en.md
    • doc/cli.md → redirected to config/cli.md
    • doc/cli.en.md → redirected to config/cli.en.md

Note: The jekyll-redirect-from plugin and doc/providers/dnscom.{md,en.md} redirect files are intentionally preserved as they use Jekyll-based redirects and are still needed for the DNS.COM → 51DNS migration.

Original prompt

删除旧的重定向文档,并检查所有markdown和yaml中是否还有错误引用


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Remove old redirect documentation and check for broken references Remove obsolete redirect docs for dnscom → 51dns migration Nov 15, 2025
Copilot AI requested a review from NewFuture November 15, 2025 09:50
@NewFuture
Copy link
Owner

@copilot keep jekyll-redirect-from and dnscom

but remove the /doc/{env,json,cli}.md

Copy link
Contributor Author

Copilot AI commented Nov 15, 2025

@copilot keep jekyll-redirect-from and dnscom

but remove the /doc/{env,json,cli}.md

Done! Reverted the dnscom changes and kept jekyll-redirect-from. Removed the /doc/{env,json,cli}.md redirect files instead (6 files total). Changes in commit b3d4a11.

Copilot AI changed the title Remove obsolete redirect docs for dnscom → 51dns migration Remove obsolete redirect docs from /doc/{env,json,cli}.md Nov 15, 2025
@NewFuture NewFuture marked this pull request as ready for review November 15, 2025 10:09
Copilot AI review requested due to automatic review settings November 15, 2025 10:09
@NewFuture
Copy link
Owner

NewFuture commented Nov 15, 2025

@copilot 再次检查所有markdown 和 yaml 是否还有对这这些删除文件的引用(索引)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes obsolete redirect stub files that were previously used to guide users from old documentation paths (/doc/) to their new locations (/doc/config/). The redirect files contained simple markdown messages directing users to update their bookmarks, but are no longer needed as all references in the codebase already point to the correct paths.

  • Removes 6 redirect stub files (Chinese and English versions of env, json, and cli documentation)
  • All existing references in the codebase already use the correct paths (doc/config/*.md)
  • The actual documentation files remain in place at doc/config/

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
doc/json.md Removed redirect stub to config/json.md
doc/json.en.md Removed English redirect stub to config/json.en.md
doc/env.md Removed redirect stub to config/env.md
doc/env.en.md Removed English redirect stub to config/env.en.md
doc/cli.md Removed redirect stub to config/cli.md
doc/cli.en.md Removed English redirect stub to config/cli.en.md

Copy link
Contributor Author

Copilot AI commented Nov 15, 2025

@copilot 再次检查所有markdown 和 yaml 是否还有对这这些删除文件的引用(索引)

Checked all markdown and YAML files. ✅ No broken references found.

All references correctly point to doc/config/{cli,env,json}.{md,en.md} files. The old doc/{cli,env,json}.{md,en.md} files have been successfully removed without breaking any links.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@NewFuture NewFuture merged commit 6cc1af8 into master Nov 15, 2025
37 checks passed
@NewFuture NewFuture deleted the copilot/remove-old-redirect-docs branch November 15, 2025 12:34
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.

2 participants