docker icon indicating copy to clipboard operation
docker copied to clipboard

Send PR for Sync Gateway Docker official README

Open tleyden opened this issue 9 years ago • 3 comments

  • [ ] Review README for marketing purposes (@jamiltz)
  • [ ] Run READE through Docker tool to check for valid markdown (@ceejatec)
  • [x] If https://github.com/docker-library/official-images/pull/2082 has already been merged, then we can change "NOTE: until the official docker image is approved, use the staging repository via:" to point to the official Docker image
  • [ ] Split the README into sections and enhance generator to cat into single README (@tleyden)
  • [ ] Push README PR

tleyden avatar Aug 23 '16 00:08 tleyden

The upstream PR got merged: https://github.com/docker-library/official-images/pull/2082

tleyden avatar Oct 26 '16 17:10 tleyden

No, it didn't actually - that was my fault, I guess. I proposed a different change (adding 4.1.1 CE) from the same git branch, and that got merged, which fooled this github PR into thinking it was merged too.

I will re-propose the sync gateway addition after making some of the suggested changes.

Ceej aka Chris Hillery

On Oct 26, 2016 10:03 AM, "Traun Leyden" [email protected] wrote:

The upstream PR got merged: docker-library/official-images#2082 https://github.com/docker-library/official-images/pull/2082

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/couchbase/docker/issues/52#issuecomment-256413331, or mute the thread https://github.com/notifications/unsubscribe-auth/ABZIw4B2BOK2Hgmneujgh5JlAk9yp_VAks5q34fYgaJpZM4Jqcz6 .

ceejatec avatar Oct 27 '16 15:10 ceejatec

Ok thanks for the clarification

tleyden avatar Oct 27 '16 17:10 tleyden