Skip to content
This repository has been archived by the owner on Jun 8, 2019. It is now read-only.

Add new field "Empty" to Repository structure #56

Merged
merged 3 commits into from
May 4, 2017
Merged

Add new field "Empty" to Repository structure #56

merged 3 commits into from
May 4, 2017

Conversation

lafriks
Copy link
Member

@lafriks lafriks commented May 4, 2017

Add new field where to return repo.IsBare boolean from Gitea API

@bkcsoft
Copy link
Member

bkcsoft commented May 4, 2017

IsBare is a very specific thing, and not to be confused with a repo being empty :)

But LGTM 👍

@lunny
Copy link
Member

lunny commented May 4, 2017

Why it's a string type not a bool?

@lafriks
Copy link
Member Author

lafriks commented May 4, 2017

ouch, sry, fixed to boolean

@lunny
Copy link
Member

lunny commented May 4, 2017

LGTM

@lunny
Copy link
Member

lunny commented May 4, 2017

No empty in github's API v3.

@guyzmo
Copy link

guyzmo commented May 4, 2017

@lunny indeed there's no empty, I use size from that API. I like the empty property, as it makes it explicit when you read the code.

@lunny lunny merged commit 59cdd51 into go-gitea:master May 4, 2017
guyzmo added a commit to guyzmo/git-repo that referenced this pull request May 4, 2017
guyzmo added a commit to guyzmo/git-repo that referenced this pull request May 4, 2017
guyzmo added a commit to guyzmo/git-repo that referenced this pull request May 4, 2017
sapk added a commit to sapk-fork/gitea that referenced this pull request May 14, 2017
@sapk sapk mentioned this pull request May 14, 2017
appleboy pushed a commit to go-gitea/gitea that referenced this pull request May 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants