Skip to content

fixed bug #151 caused Find should be Get #153

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

Merged
merged 1 commit into from
Nov 11, 2016

Conversation

lunny
Copy link
Member

@lunny lunny commented Nov 11, 2016

When get some one object, use Get. When get a collection of objects, use Find.

@lunny lunny added this to the 1.0.0 milestone Nov 11, 2016
@lunny lunny added the type/bug label Nov 11, 2016
@codecov-io
Copy link

codecov-io commented Nov 11, 2016

Current coverage is 3.06% (diff: 0.00%)

Merging #153 into master will increase coverage by 0.02%

@@            master      #153   diff @@
========================================
  Files           33        33          
  Lines         8096      8100     +4   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
+ Hits           246       248     +2   
- Misses        7830      7833     +3   
+ Partials        20        19     -1   

Powered by Codecov. Last update 9bf28a2...ec7b136

@tboerger
Copy link
Member

LGTM

1 similar comment
@thibaultmeyer
Copy link
Contributor

LGTM

@thibaultmeyer thibaultmeyer merged commit 555d8b1 into go-gitea:master Nov 11, 2016
lunny added a commit to lunny/gitea that referenced this pull request Nov 12, 2016
@lunny lunny mentioned this pull request Nov 12, 2016
@lunny lunny deleted the lunny/bug_fixed_151 branch November 12, 2016 00:17
thibaultmeyer pushed a commit that referenced this pull request Nov 12, 2016
@tboerger tboerger added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 29, 2016
@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/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants