WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@Layau-code
Copy link
Contributor

修复 README 表格的 markdownlint 问题(MD060/MD009),将表格改为仓库要求的 compact 风格并删除行尾多余空格,使 document-lint CI 通过。

@LearningGp LearningGp added the kind/bug Category issues or prs related to bug. label Dec 16, 2025
@LearningGp LearningGp moved this to In progress in Sentinel Dec 16, 2025
@LearningGp LearningGp moved this from In progress to In review in Sentinel Dec 16, 2025
@LearningGp
Copy link
Collaborator

看起来还有一些markdownlint MD060以及MD009问题,方便的话帮忙一起在这个PR修复下

@Layau-code
Copy link
Contributor Author

好的,我会继续尝试修复

@Layau-code Layau-code force-pushed the fix/docs/parameter-flow-table branch from 32e0a3f to 4412b35 Compare December 16, 2025 07:12
@Layau-code
Copy link
Contributor Author

@LearningGp 你好!

我已经修复了你提到的 markdownlint MD060 和 MD009 问题。具体修复内容:

  1. sentinel-parameter-flow-control/README.md:修复了表格格式,将压缩在一行的表格拆分为标准 Markdown 格式
  2. sentinel-spring-webmvc-adapter/README.md:修复了 3 个配置表格的格式问题
  3. sentinel-spring-webmvc-v6x-adapter/README.md:移除了表格中的多余空格

我在本地验证了一下,没有什么问题,希望可以重新review,如果还有问题我会继续做出修改。

@LearningGp
Copy link
Collaborator

看起来其他的都没问题了,还有两个文件存在 MD034/MD060的问题

@Layau-code
Copy link
Contributor Author

好的,那我再改一下这两个文件

@Layau-code Layau-code force-pushed the fix/docs/parameter-flow-table branch 2 times, most recently from 4137f0f to 8c232bb Compare December 16, 2025 08:34
- Restore correct UTF-8 encoding for all Chinese text
- Fix garbled Chinese in company logos (拼多多, 爱奇艺, 二维火, etc.)
- Fix table column style (MD060) - remove spaces around pipes
- Fix bare URLs (MD034) - wrap URLs with angle brackets
- Remove BOM markers from documentation files

Fixes encoding issues from commit 7b9f7c7
Disabled core.autocrlf to prevent encoding corruption
@Layau-code Layau-code force-pushed the fix/docs/parameter-flow-table branch from 8c232bb to 465f6dc Compare December 16, 2025 08:49
- Fix bare URL in Sentinel_Dashboard_Feature.md (控制台 Wiki link)
- Fix bare URLs in sentinel-demo-dubbo README (流量控制, 控制台文档)

All markdownlint MD034 errors now resolved
@Layau-code
Copy link
Contributor Author

@LearningGp 你好!

MD034/MD060的问题已经修复了

- Fix trailing spaces in sentinel-adapter/sentinel-spring-webmvc-adapter/README.md
- Fix trailing spaces in sentinel-cluster/sentinel-cluster-server-envoy-rls/README.md
- Fix trailing spaces in sentinel-dashboard/README.md

All MD009 errors now resolved
@Layau-code
Copy link
Contributor Author

Layau-code commented Dec 16, 2025

这次是MD009的问题,我已经修复了,可以再次review!

@Layau-code
Copy link
Contributor Author

好像超时了
是我的问题吗

Copy link
Collaborator

@LearningGp LearningGp left a comment

Choose a reason for hiding this comment

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

LGTM

@LearningGp LearningGp merged commit e7a9c56 into alibaba:1.8 Dec 18, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Sentinel Dec 18, 2025
@Layau-code Layau-code deleted the fix/docs/parameter-flow-table branch December 18, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Category issues or prs related to bug.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants