Skip to content

Conversation

@carlaKC
Copy link
Contributor

@carlaKC carlaKC commented Mar 28, 2020

Rename le project of many names to faraday.
Changes isolated to a single commit, just search-and-replaced everything + regenerated protos.

@carlaKC carlaKC requested a review from guggero March 28, 2020 12:40
Copy link

@cfromknecht cfromknecht left a comment

Choose a reason for hiding this comment

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

just some drive by comments :)

README.md Outdated
[![MIT licensed](https://img.shields.io/badge/license-MIT-blue.svg)](https://github.com/lightninglabs/faraday/blob/master/LICENSE)

The governator is an external service intended to be run in conjunction with the [lnd](https://github.com/lightningnetwork/lnd) implementation of the [Lightning Network](https://lightning.network). It queries LND for information about its existing channels and provides channel close recommendations if channels are under-performing.
The faraday is an external service intended to be run in conjunction with the [lnd](https://github.com/lightningnetwork/lnd) implementation of the [Lightning Network](https://lightning.network). It queries LND for information about its existing channels and provides channel close recommendations if channels are under-performing.

Choose a reason for hiding this comment

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

"The" no longer necessary?

README.md Outdated

## Installation
A [Makefile](https://github.com/lightninglabs/governator/blob/master/Makefile) is provided. To install governator and al its dependencies, run:
A [Makefile](https://github.com/lightninglabs/faraday/blob/master/Makefile) is provided. To install faraday and al its dependencies, run:

Choose a reason for hiding this comment

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

al -> all

Copy link
Contributor

@guggero guggero left a comment

Choose a reason for hiding this comment

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

Nice, I like the name!
Found two forgotten references, looks good to me otherwise 💯

frdrpc/README.md Outdated
@@ -1,8 +1,8 @@
gvrpc
Copy link
Contributor

Choose a reason for hiding this comment

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

s/gvrpc/frdrpc/

@@ -1,14 +1,14 @@
// Package gvrpc contains the proto files, generated code and server logic
Copy link
Contributor

Choose a reason for hiding this comment

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

s/gvrpc/frdrpc

@carlaKC carlaKC merged commit 2c449ab into master Mar 30, 2020
@carlaKC carlaKC deleted the rename-faraday branch April 21, 2020 11:34
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.

4 participants