tuxagon

Emoji Code Review Comments

Legend

Emoji	Translation
😃 😍	I like this! No changes or acknowledgements needed. Just wanted to say well done.
❌ ⚠️	Problem This is a blocking issue and requires changes.
🔧 🎨	Suggestion Not blocking, but a suggestion or idea for improvement. Feel free to disagree and move on.
❓	QuestionNot blocking, but requires an answer by the PR/code author.

tuxagon

#!/bin/bash

# Generate a `:something-intensifies:` Slack emoji, given a reasonable image
# input. I recommend grabbing an emoji from https://emojipedia.org/

set -euo pipefail

# Number of frames of shaking
count=10
# Max pixels to move while shaking

tuxagon

# Build image with Node.js, Elm and Nginx
FROM debian:latest

# prepare
RUN apt-get -y update
RUN apt-get -y install apt-utils

# install curl, http://stackoverflow.com/questions/27273412/cannot-install-packages-inside-docker-ubuntu-image
RUN apt-get -y install curl

tuxagon

package fakeresponse

import (
  "testing"
  "net/http"
)

type FakeResponse struct {
  t *testing.T
  headers http.Header

tuxagon

package fakeresponse

import (
  "testing"
  "net/http"
)

type FakeResponse struct {
  t *testing.T
  headers http.Header

tuxagon

package fakeresponse

import (
  "testing"
  "net/http"
)

type FakeResponse struct {
  t *testing.T
  headers http.Header

tuxagon

package tests

import (
  "testing"
)

type S struct {
  t *testing.T
}