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

Implement CLI history command #4702

Merged
merged 1 commit into from
Nov 9, 2015
Merged

Implement CLI history command #4702

merged 1 commit into from
Nov 9, 2015

Conversation

pires
Copy link
Contributor

@pires pires commented Nov 7, 2015

  • CHANGELOG.md updated
  • Rebased/mergeable
  • Tests pass
  • Sign CLA (if not already signed)

Pay special attention to the a couple facts:

Now, the output:

$ influx -host 192.168.99.100
Visit https://enterprise.influxdata.com to register for updates, InfluxDB server management, and monitoring.
Connected to http://192.168.99.100:8086 version 0.9.3
InfluxDB shell 0.9
> history
help
history
help
history
show databases
show tag values
help
history
x
history
x
history
>

Fixes #4628

@corylanou
Copy link
Contributor

+1

1 similar comment
@otoolep
Copy link
Contributor

otoolep commented Nov 9, 2015

+1

otoolep added a commit that referenced this pull request Nov 9, 2015
@otoolep otoolep merged commit fbfa175 into influxdata:master Nov 9, 2015
@otoolep
Copy link
Contributor

otoolep commented Nov 9, 2015

Looks good, thanks @pires - this change exposed 1 minor issue with history in general, for which I opened #4719.

@pires pires deleted the 4628-cli_history branch November 9, 2015 23:52
@sdtsui sdtsui mentioned this pull request Sep 11, 2016
4 tasks
sdtsui added a commit to sdtsui/influxdb that referenced this pull request Sep 13, 2016
- Single commit, PR follows conventions laid out by @gouthamve in influxdata#5822
* main.go: struct field CpuFile should be CPUFile
* influx_inspect: loop equivalent to `for key := range...`
* adds comments to exported fields and consts
* fixes typo in `CHANGELOG.md`: text for influxdata#4702 now matches number
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

3 participants