Skip to content

Conversation

gabrielsimoes
Copy link
Contributor

@gabrielsimoes gabrielsimoes commented Jan 25, 2019

Signed-off-by: Gabriel Silva Simões simoes.sgabriel@gmail.com

Signed-off-by: Gabriel Silva Simões <simoes.sgabriel@gmail.com>
@gabrielsimoes gabrielsimoes mentioned this pull request Jan 25, 2019
@codecov-io
Copy link

codecov-io commented Jan 25, 2019

Codecov Report

Merging #5836 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5836      +/-   ##
==========================================
- Coverage   37.85%   37.85%   -0.01%     
==========================================
  Files         328      328              
  Lines       48326    48326              
==========================================
- Hits        18294    18292       -2     
- Misses      27399    27401       +2     
  Partials     2633     2633
Impacted Files Coverage Δ
models/repo_list.go 63.29% <0%> (-1.27%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 182a6d6...c22fc8a. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 25, 2019
@zeripath
Copy link
Contributor

Add a test with reference to #2167 in the comments so that this doesn't happen again

Signed-off-by: Gabriel Silva Simões <simoes.sgabriel@gmail.com>
@gabrielsimoes
Copy link
Contributor Author

Tests added.

@techknowlogick techknowlogick added this to the 1.8.0 milestone Jan 26, 2019
gabrielsimoes and others added 2 commits January 26, 2019 13:24
Change 0/1 to false/true and remove repetitive code

Signed-off-by: Gabriel Silva Simões <simoes.sgabriel@gmail.com>
@lafriks lafriks added the topic/ui Change the appearance of the Gitea UI label Jan 27, 2019
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jan 27, 2019
@GiteaBot GiteaBot 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 Jan 27, 2019
@zeripath zeripath merged commit 5f4a40c into go-gitea:master Jan 27, 2019
bmackinney pushed a commit to bmackinney/gitea that referenced this pull request Jan 27, 2019
…admins go-gitea#2167 (go-gitea#5836)

Revert go-gitea#3711 overwrite of only show "No Description" to repo admins  go-gitea#2167 and add tests to prevent accidental overwrite again.
@gabrielsimoes gabrielsimoes deleted the revert-3711 branch January 30, 2019 14:25
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
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. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants