Skip to content
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

cli: detect bareness of git libraries #897

Merged
merged 4 commits into from
Jun 21, 2019

Conversation

jfontan
Copy link
Contributor

@jfontan jfontan commented Jun 21, 2019

Now by default the type of git repositories (bare or non bare) is chosen automatically using the first repository found in the directory. To force an specific format --bare and --non-bare can be used. Also the URL format now has 3 possible values for bare option: true, false and auto.

$ gitbase server --non-bare -d //dir/1
$ gitbase server -d 'file:///dir/1?bare=auto' -d 'file:///dir/1?bare=true'
  • I updated the documentation explaining the new behavior if any.
  • I updated CHANGELOG.md file adding the new feature or bug fix.
  • I updated go-mysql-server using make upgrade command if applicable.
  • I added or updated examples if applicable.
  • I checked that changes on schema are reflected into the documentation, if applicable.

@jfontan jfontan requested review from smola and a team June 21, 2019 09:57
cmd/gitbase/command/server_test.go Outdated Show resolved Hide resolved
cmd/gitbase/command/server.go Outdated Show resolved Hide resolved
cmd/gitbase/command/server.go Outdated Show resolved Hide resolved
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
Signed-off-by: Javi Fontan <jfontan@gmail.com>
@jfontan
Copy link
Contributor Author

jfontan commented Jun 21, 2019

I've rebased with latest master

Signed-off-by: Javi Fontan <jfontan@gmail.com>
@ajnavarro ajnavarro merged commit 42ec285 into src-d:master Jun 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants