Updated readme, fix incorrect newline in docker start.sh, add example usage #53

Merged
prologic merged 12 commits from brokenscripts/golinks:master into master 3 days ago
  1. Line 23 in start.sh (Docker) contained an erroneous newline that would cause the container to constantly restart. Removed that.

  2. Updated the README.md to contain the current usage (0.0.7 + commits) of golinks.

    • Added an updated compose/docker section NOTE I am using my repo brokenscripts/golinks on DockerHub to align with 0.0.7 until your latest version is pushed.
    • Updated the configuration section to show the actual variables used in the current release.
  3. Added a small gif to show usage.


Files Updated:

  • README.md
  • start.sh

Files Added:

  • golinks.gif/golinks.gif
1. Line 23 in `start.sh` (Docker) contained an erroneous newline that would cause the container to constantly restart. Removed that. 2. Updated the README.md to contain the current usage (0.0.7 + commits) of golinks. * Added an updated compose/docker section **NOTE** I am using my repo `brokenscripts/golinks` on DockerHub to align with 0.0.7 until your latest version is pushed. * Updated the configuration section to show the actual variables used in the current release. 3. Added a small gif to show usage. --- Files Updated: * `README.md` * `start.sh` Files Added: * `golinks.gif/golinks.gif`
brokenscripts added 6 commits 7 days ago
brokenscripts changed title from WIP: Updated readme, fix incorrect newline in docker start.sh, add example usage to Updated readme, fix incorrect newline in docker start.sh, add example usage 7 days ago
Poster

Not sure why it also pushed #52 again. In my original post I put the 3 files modified/added.

Not sure why it also pushed #52 again. In my original post I put the 3 files modified/added.
prologic approved these changes 6 days ago
Owner

@brokenscripts Do you mind fixing the conflicts? Otherwise changes looks good.

@brokenscripts Do you mind fixing the conflicts? Otherwise changes looks good.
Owner

@brokenscripts Do you mind fixing the conflicts? Otherwise changes looks good.

@brokenscripts Do you mind fixing the conflicts? Otherwise changes looks good.
brokenscripts added 6 commits 6 days ago
Poster

@prologic Done. Had to do some google-fu on how to make the merge conflicts go away after a pull request was accepted. Anything else?

@prologic Done. Had to do some google-fu on how to make the merge conflicts go away after a pull request was accepted. Anything else?
Owner

@prologic Done. Had to do some google-fu on how to make the merge conflicts go away after a pull request was accepted. Anything else?

Thanks!

> @prologic Done. Had to do some google-fu on how to make the merge conflicts go away after a pull request was accepted. Anything else? Thanks!
prologic merged commit ef007105dd into master 3 days ago

Reviewers

prologic approved these changes 6 days ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as ef007105dd.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.