search

hgomez / influxdb

InfluxDB Tools
19 stars 2 forks source link

Improve documentation #5

Closed amotl closed 3 years ago

amotl commented 3 years ago

Dear Henri,

thanks for integrating #4. This patch adds some improvements to the documentation and, through bf980d27, some convenience for users to invoke the program instead of having to address the .jar file.

With kind regards, Andreas.

hgomez commented 3 years ago

I won't use this URL in setup https://raw.githubusercontent.com/daq-tools/influxdb-fetcher/develop/bin/influxdb-fetcher section

amotl commented 3 years ago

I won't use this URL in setup.

True, thanks! Can you tag your repository with 1.0.1 and then add the build influxdb-fetcher-1.0.1.jar as an artifact/asset to the GitHub release - similar to what I did at [1]? Then, I can amend this patch accordingly.

[1] https://github.com/daq-tools/influxdb-fetcher/releases

amotl commented 3 years ago

Maybe also rename the repository itself to influxdb-fetcher? No worries about the "Cool URIs don't change" thing. GitHub will automatically redirect people accessing the repository to the new location - both on the Web side as well as the Git side of things.

hgomez commented 3 years ago

I released 1.0.2, and included jar Let's keep influxdb as repository name, more tools may surface later

amotl commented 3 years ago

I released 1.0.2, and included jar.

Thanks.

Let's keep influxdb as repository name, more tools may surface later.

All right.

I've just amended this patch to make the URLs match your repository layout again.

hgomez commented 3 years ago

Thanks Andreas

amotl commented 3 years ago

Thank you as well, Henri. I verified that the installation procedure works and then added a comment at https://github.com/influxdata/influxdb/issues/3904#issuecomment-771031435 in order to reflect the recent improvements.