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.

Sign up for GitHub

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

Jump to bottom

Rewrite a paragraph in the README and a couple trivial fixes #13

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Rewrite a paragraph in the README and a couple trivial fixes #13

wants to merge 3 commits into from

Commits on Mar 25, 2022

  1. campaign: rename error variable to aqErr 

    The error for the switch covering most of the body of the main loop of
Acquire shouldn't be called `err`. Give it a better name.
    @dfinkel
    dfinkel committed Mar 25, 2022
    Configuration menu
    Copy the full SHA
    00a4da4 View commit details
    Browse the repository at this point in the history

  2. gcs: use the + for concatenattion in Error() 

    Don't use fmt.Sprintf when you're just concatenating two strings.
    @dfinkel
    dfinkel committed Mar 25, 2022
    Configuration menu
    Copy the full SHA
    5b94267 View commit details
    Browse the repository at this point in the history

  3. Readme: reword a paragraph so it's not misleading 

    The description of OnElected, OnOusting, and LeaderChanged are a bit
misleading because of an interstitial clause. Rewrite that paragraph and
break it up into two.
    @dfinkel
    dfinkel committed Mar 25, 2022
    Configuration menu
    Copy the full SHA
    dbf3acb View commit details
    Browse the repository at this point in the history