Page MenuHomePhabricator

Update `.gitignore`.
ClosedPublic

Authored by joshuaspence on Jun 14 2014, 6:18 PM.
Tags
None
Referenced Files
F13429378: D9527.diff
Wed, Jul 10, 3:26 AM
F13414822: D9527.id.diff
Sun, Jul 7, 3:49 PM
F13406547: D9527.diff
Fri, Jul 5, 9:34 PM
F13395065: D9527.id22835.diff
Wed, Jul 3, 12:19 PM
F13395064: D9527.id22844.diff
Wed, Jul 3, 12:19 PM
F13395063: D9527.id.diff
Wed, Jul 3, 12:19 PM
F13395062: D9527.id22841.diff
Wed, Jul 3, 12:19 PM
F13395061: D9527.id22834.diff
Wed, Jul 3, 12:19 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rPadbdea8d3f90: Update `.gitignore`.
Summary

See inline comments for reasoning.

Test Plan

N/A

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Update `.gitignore`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
.gitignore
1–12

These files are specific to the developer's environment. They should be include in the developer's local gitignore.

14–15

I don't think that this exists anymore.

joshuaspence edited edge metadata.

/.divinercache is a directory.

epriestley edited edge metadata.
  • In theory, you are completely correct.
  • In practice, we get pull requests if we don't put this stuff in .gitignore. This didn't seem like the most important battle to fight after a couple of them.

I'll accept this if you put a message in its place, like:

# NOTE: Thinking about adding files created by your operating system, IDE,
# or text editor here? Don't! Add them to your per-user .gitignore instead.

..and we can see how that goes?

This revision now requires changes to proceed.Jun 14 2014, 6:36 PM
joshuaspence edited edge metadata.
  • Adding a comment as requested.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 14 2014, 6:44 PM
epriestley updated this revision to Diff 22844.

Closed by commit rPadbdea8d3f90 (authored by @joshuaspence, committed by @epriestley).