Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(246)

Issue 44120044: Updating sample for video. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by paulrashidi
Modified:
10 years, 4 months ago
Reviewers:
danielholevoet
CC:
douga
Visibility:
Public.

Description

Updating sample for video.

Patch Set 1 #

Total comments: 15

Patch Set 2 : Gradle version upgrade. README update including Client ID explanation. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+134 lines, -112 lines) Patch
M .gitignore View 1 1 chunk +1 line, -2 lines 0 comments Download
M HelloEndpointsProject/HelloEndpoints/build.gradle View 1 2 chunks +4 lines, -4 lines 0 comments Download
M HelloEndpointsProject/gradle/wrapper/gradle-wrapper.jar View 0 chunks +-1 lines, --1 lines 0 comments Download
M HelloEndpointsProject/gradle/wrapper/gradle-wrapper.properties View 1 chunk +2 lines, -2 lines 0 comments Download
A HelloEndpointsProject/gradlew View 0 chunks +-1 lines, --1 lines 0 comments Download
M HelloEndpointsProject/gradlew.bat View 1 chunk +90 lines, -90 lines 0 comments Download
M README.md View 1 3 chunks +39 lines, -16 lines 0 comments Download

Messages

Total messages: 6
paulrashidi
10 years, 4 months ago (2013-12-19 00:34:01 UTC) #1
danielholevoet
https://codereview.appspot.com/44120044/diff/1/HelloEndpointsProject/HelloEndpoints/.gitignore File HelloEndpointsProject/HelloEndpoints/.gitignore (right): https://codereview.appspot.com/44120044/diff/1/HelloEndpointsProject/HelloEndpoints/.gitignore#newcode1 HelloEndpointsProject/HelloEndpoints/.gitignore:1: /build A single, top-level file with HelloEndpoints/build in it ...
10 years, 4 months ago (2013-12-19 01:19:44 UTC) #2
paulrashidi
Updating sample before recording video.
10 years, 4 months ago (2013-12-19 03:23:40 UTC) #3
paulrashidi
I removed individual .gitignore files but that is how net new projects work in AndroidStudio. ...
10 years, 4 months ago (2013-12-19 03:26:27 UTC) #4
danielholevoet
LGTM
10 years, 4 months ago (2013-12-19 17:41:59 UTC) #5
paulrashidi
10 years, 4 months ago (2013-12-19 20:01:49 UTC) #6

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b