Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1028)

Issue 515014: Add a basic .gitignore file for the autotest repo. (Closed)

Created:
11 years ago by petkov
Modified:
9 years, 7 months ago
Reviewers:
ericli
CC:
chromium-os-reviews_googlegroups.com
Visibility:
Public.

Description

Add a basic .gitignore file for the autotest repo.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Filter results.* instead. #

Total comments: 1

Patch Set 3 : changed to results.*/ to cover results.*/ folders anywhere #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -0 lines) Patch
A .gitignore View 1 2 1 chunk +17 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
petkov
11 years ago (2009-12-23 18:24:39 UTC) #1
ericli
http://codereview.chromium.org/515014/diff/1/2 File .gitignore (right): http://codereview.chromium.org/515014/diff/1/2#newcode16 .gitignore:16: results/ better include results.* by default the results directory ...
11 years ago (2009-12-23 18:51:12 UTC) #2
petkov
On 2009/12/23 18:51:12, ericli wrote: > http://codereview.chromium.org/515014/diff/1/2 > File .gitignore (right): > > http://codereview.chromium.org/515014/diff/1/2#newcode16 > ...
11 years ago (2009-12-23 20:24:33 UTC) #3
ericli
http://codereview.chromium.org/515014/diff/3001/3002 File .gitignore (right): http://codereview.chromium.org/515014/diff/3001/3002#newcode16 .gitignore:16: results/results.*/ no, should be: results results.* if you dont ...
11 years ago (2009-12-23 20:36:01 UTC) #4
petkov
11 years ago (2009-12-23 20:43:10 UTC) #5
ericli
11 years ago (2009-12-23 20:44:39 UTC) #6
On 2009/12/23 20:43:10, petkov wrote:
> 

LGTM.

Powered by Google App Engine
This is Rietveld 408576698