Skip to content

Keep same repo.ID on repo file updates #3414

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.

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

Closed
wants to merge 2 commits into from
Closed

Keep same repo.ID on repo file updates #3414

wants to merge 2 commits into from

Conversation

thehowl
Copy link
Contributor

@thehowl thehowl commented Jan 26, 2018

I really hope for my sanity's sake that this breaks because otherwise I have no idea why this is happening.

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@82a7b02). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3414   +/-   ##
=========================================
  Coverage          ?   35.66%           
=========================================
  Files             ?      281           
  Lines             ?    40553           
  Branches          ?        0           
=========================================
  Hits              ?    14464           
  Misses            ?    23957           
  Partials          ?     2132
Impacted Files Coverage Δ
models/repo_editor.go 14.2% <100%> (ø)

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 82a7b02...21a1553. Read the comment docs.

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 26, 2018
@sapk
Copy link
Member

sapk commented Jan 26, 2018

Is this related to #3386 ?

@tboerger tboerger 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 26, 2018
@thehowl thehowl closed this Jan 26, 2018
@thehowl thehowl deleted the fix-drone-1 branch January 26, 2018 22:38
@thehowl
Copy link
Contributor Author

thehowl commented Jan 26, 2018

As we found out in #develop, this is actually not the issue at hand, so nevermind.

@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.
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants