Skip to content

Conversation

@pellared
Copy link
Member

@pellared pellared commented May 3, 2020

Review: https://github.com/golang-templates/library/blob/8495f35f9d6adb09c16e9709307bee7a77160b00/README.md

README refined.
New FAQ section:

  • Why Mage instead of Make
  • Why nothing for GoLand
  • Why GitHub Actions, not any other CI server

CircleCI config works: https://app.circleci.com/pipelines/github/pellared/test-circleci/1/workflows/8236f2e3-3f39-443a-80b7-994b93b3da5b/jobs/1

Makefile also tested

@pellared pellared marked this pull request as ready for review May 3, 2020 09:04
@pellared pellared requested a review from agabrys May 3, 2020 11:33
@pellared pellared added the documentation Improvements or additions to documentation label May 3, 2020
Copy link
Contributor

@agabrys agabrys left a comment

Choose a reason for hiding this comment

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

I found some issues (typos, formatting inconsistency). I added also some suggestions (I don't like configs...).

Squash your changes and add a nice describing commit message (header + description), please.

pellared and others added 12 commits May 3, 2020 14:19
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
@pellared pellared requested a review from agabrys May 3, 2020 12:42
Copy link
Contributor

@agabrys agabrys left a comment

Choose a reason for hiding this comment

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

Two more findings.

pellared and others added 2 commits May 3, 2020 18:06
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Co-authored-by: Adam Gabryś <adam.gabrys@live.com>
Copy link
Contributor

@agabrys agabrys left a comment

Choose a reason for hiding this comment

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

Squash and merge 🙂

@pellared pellared merged commit 0cc107b into master May 3, 2020
@pellared pellared deleted the refinements branch May 3, 2020 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants