Skip to content
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

Sync MaxGitDiffLineCharacters with conf/app.ini (#2779) #2780

Merged
merged 1 commit into from
Oct 25, 2017
Merged

Sync MaxGitDiffLineCharacters with conf/app.ini (#2779) #2780

merged 1 commit into from
Oct 25, 2017

Conversation

michaelkuhn
Copy link
Contributor

Backport of #2779

@michaelkuhn
Sync MaxGitDiffLineCharacters with conf/app.ini (#2779)
cc9b4d8 
MAX_GIT_DIFF_LINE_CHARACTERS was updated in #1845 but the corresponding
default value of MaxGitDiffLineCharacters was not changed. This can lead
to inconsistencies.
@lafriks lafriks added the issue/regression Issue needs no code to be fixed, only a description on how to fix it yourself label Oct 25, 2017
@lafriks lafriks added this to the 1.2.2 milestone Oct 25, 2017
@lafriks
Copy link
Member

lafriks commented Oct 25, 2017

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Oct 25, 2017
@lunny
Copy link
Member

lunny commented Oct 25, 2017

LGTM

@lafriks
Copy link
Member

lafriks commented Oct 25, 2017

Make LG-TM work

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Oct 25, 2017
@lafriks lafriks merged commit f40ba68 into go-gitea:release/v1.2 Oct 25, 2017
@lafriks
Copy link
Member

lafriks commented Oct 25, 2017

@michaelkuhn thanks

@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
issue/regression Issue needs no code to be fixed, only a description on how to fix it yourself lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore.
Projects
None yet
Milestone
1.2.2
Development

Successfully merging this pull request may close these issues.
4 participants
@michaelkuhn @lafriks @lunny @tboerger