Skip to content

Print go version along with Dgraph version.#2768

Merged
manishrjain merged 1 commit intomasterfrom
danielmai/print_go_version
Nov 26, 2018
Merged

Print go version along with Dgraph version.#2768
manishrjain merged 1 commit intomasterfrom
danielmai/print_go_version

Conversation

@danielmai
Copy link
Copy Markdown
Contributor

@danielmai danielmai commented Nov 21, 2018 •
edited
Loading

Helpful to know for things like custom tokenizers.

Example output:

$ ./dgraph version

Dgraph version   : v1.0.11-rc3-3-g57ac1cd4
Commit SHA-1     : 57ac1cd4
Commit timestamp : 2018-11-21 14:58:30 -0800
Branch           : danielmai/print_go_version
Go version       : go1.10.1

For Dgraph official documentation, visit https://docs.dgraph.io.
For discussions about Dgraph     , visit https://discuss.dgraph.io.
To say hi to the community       , visit https://dgraph.slack.com.

Licensed under Apache 2.0. Copyright 2015-2018 Dgraph Labs, Inc.

This change is Reviewable

manishrjain
manishrjain approved these changes Nov 22, 2018
View reviewed changes
Copy link
Copy Markdown
Contributor

@manishrjain manishrjain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @manishrjain)

@manishrjain manishrjain merged commit 50eb866 into master Nov 26, 2018
@manishrjain manishrjain deleted the danielmai/print_go_version branch November 26, 2018 07:35
dna2github pushed a commit to dna2fork/dgraph that referenced this pull request Jul 19, 2019
@danielmai @dna2github
Print go version along with Dgraph version. (dgraph-io#2768)
6027c7a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@manishrjain manishrjain manishrjain approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@danielmai @manishrjain