8000 A few minor fixes to review by flaix · Pull Request #9 · flaix/gitblit · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

A few minor fixes to review #9

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
Closed
wants to merge 11 commits into from
Closed

A few minor fixes to review #9

wants to merge 11 commits into from

Conversation

flaix
Copy link
Owner
@flaix flaix commented Nov 7, 2019

Buried within this merge request seemingly lie a few minor fixes worth checking out.

yumine-cc added 11 commits August 17, 2017 11:37
@yumine-cc
create spring-boot branch
844ae09
@yumine-cc
create maven pom.xml
8ad96c2
@yumine-cc
增加了SpringBoot相关的实现与启动类，同时增加了用于临时修复包含有特殊字符的路径不能正确解译的热补丁。待完成测试后再正式合并。
d6765e1
@yumine-cc
增强了Raw模式下对于各类文件在浏览器中的输出（仅在浏览不支持文件预览的情况下，才触发下载过程）
8333c52
@yumine-cc
1, 修复了BlamePage中在处理author时出现的空异常.
4865c5c 
2，修复了Docs页面在没有找到文件时返回内部错误的问题。
@yumine-cc
修复了用户偏好中语言面板无法正确的匹配locale前缀的问题
76066ae
@yumine-cc
当commit.author为空时，使用commit.email替代。
18dcf5e
@yumine-cc
修复了Gitblit Banner的控制台输出
081e857
@yumine-cc
Merge remote-tracking branch 'origin/master' into gitblit-spring-boot
5a5e73d
@yumine-cc
升级lucene从4.10.4到5.5.2
9f101ea
@yumine-cc
对Eclipse的工程配置进行忽略处理
0341396
flaix
flaix commented Nov 11, 2019
View reviewed changes
Copy link
Owner Author
@flaix flaix left a comment •
edited
Loading

Choose a reason for hiding this comment

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

18dcf5e

I wonder what the use case for this is. I was unable to fabricate a Git setting, where a user.name would be only blanks caught by the isWhitespace method.

flaix
flaix commented Nov 11, 2019
View reviewed changes
Copy link
Owner Author
@flaix flaix left a comment •
edited
Loading

Choose a reason for hiding this comment

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

4865c5c

The first hunk looks wrong.

The second hunk was integrated.

flaix
flaix commented Nov 11, 2019
View reviewed changes
Copy link
Owner Author
@flaix flaix left a comment

Choose a reason for hiding this comment

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

76066ae

Nice catch.
Integrated, but fixed differently, as there was a problem with the search for multiple locale strings and the same problem exists on the EditUserPage.

@flaix
Copy link
Owner Author
flaix commented Nov 11, 2019

The good parts integrated with 96a7e6e in gitblit-org#1324.

@flaix flaix closed this Nov 11, 2019
< 5CB7 /option>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@flaix @yumine-cc
0