Skip to content

config lgtm to disable maintainer to LGTM himself's PR#23

Merged
tboerger merged 2 commits intogo-gitea:developfrom
lunny:develop
Nov 3, 2016
Merged

config lgtm to disable maintainer to LGTM himself's PR#23
tboerger merged 2 commits intogo-gitea:developfrom
lunny:develop

Conversation

@lunny
Copy link
Member

@lunny lunny commented Nov 3, 2016

No description provided.

@tboerger
Copy link
Member

tboerger commented Nov 3, 2016

LGTM

1 similar comment
@xinity
Copy link
Contributor

xinity commented Nov 3, 2016

LGTM

@tboerger
Copy link
Member

tboerger commented Nov 3, 2016

Still LGTM

@tboerger tboerger merged commit e4fe693 into go-gitea:develop Nov 3, 2016
@tboerger tboerger added the type/enhancement An improvement of existing functionality label Nov 3, 2016
@tboerger tboerger added this to the 1.0.0 milestone Nov 3, 2016
@tboerger tboerger added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 29, 2016
lunny referenced this pull request in lunny/gitea Feb 7, 2019
* Implementation of Folder Jumping
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
SupenBysz added a commit to SupenBysz/gitea that referenced this pull request Nov 30, 2025
解决 Issue go-gitea#23:Wiki 页面列表中的链接后缀多了 .-

问题原因:
- 当页面标题包含连字符(如 "Ecosystem-Components")时
- 系统使用 .- 后缀来区分原始连字符和空格转换的连字符
- 导致 URL 显示为 /wiki/Ecosystem-Components.-

修复方案:
1. 添加 WebPathToCleanURLPath 函数,生成不带 .- 后缀的干净 URL
2. 添加 WebPathToGitPathWithDashMarker 函数,用于 fallback 查找
3. 修改 wikiEntryByName 增加 fallback 机制,尝试查找带 .- 的文件
4. 修改 WikiPages 和 renderViewPage 使用干净 URL

Fixes go-gitea#23

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants