Skip to content

Conversation

@HarshitOnGitHub
Copy link
Contributor

This PR includes a bug fix as well as a small refactor.

Fixes: #2314.

@codecov-io
Copy link

codecov-io commented Jan 2, 2019

Codecov Report

Merging #5622 into master will increase coverage by <.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5622      +/-   ##
==========================================
+ Coverage   37.81%   37.82%   +<.01%     
==========================================
  Files         322      322              
  Lines       47474    47471       -3     
==========================================
+ Hits        17954    17957       +3     
+ Misses      26933    26926       -7     
- Partials     2587     2588       +1
Impacted Files Coverage Δ
models/action.go 60.32% <33.33%> (+0.85%) ⬆️

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 0de57fd...8660b18. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 2, 2019
@techknowlogick techknowlogick added this to the 1.8.0 milestone Jan 2, 2019
Adds a small check to close the issues only if the referencing commits
are on the default branch.

Fixes: go-gitea#2314.
@bkcsoft bkcsoft 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 3, 2019
@bkcsoft bkcsoft 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 4, 2019
@lafriks lafriks merged commit 9f476b8 into go-gitea:master Jan 4, 2019
@lafriks lafriks changed the title issue: Don't close issues via commits on non-default branch. Don't close issues via commits on non-default branch. Jan 4, 2019
@lafriks
Copy link
Member

lafriks commented Jan 4, 2019

please send commit 9f476b8 backport to release/v1.7 branch

HarshitOnGitHub added a commit to HarshitOnGitHub/gitea that referenced this pull request Jan 5, 2019
Adds a small check to close the issues only if the referencing commits
are on the default branch.

Fixes: go-gitea#2314.
lafriks pushed a commit that referenced this pull request Jan 5, 2019
Adds a small check to close the issues only if the referencing commits
are on the default branch.

Fixes: #2314.
@techknowlogick techknowlogick added the backport/done All backports for this PR have been created label Jan 13, 2019
@lunny lunny added the pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! label Feb 6, 2019
@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

backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

closing issues via commit on non-default branch

7 participants