Page MenuHomePhabricator

Update `.gitignore`.
ClosedPublic

Authored by joshuaspence on Jun 14 2014, 6:18 PM.
Tags
None
Referenced Files
F19825209: D9527.id22835.diff
Sun, Mar 8, 8:57 AM
F19824275: D9527.id22835.diff
Sat, Mar 7, 9:04 PM
F19814901: D9527.id22841.diff
Wed, Mar 4, 4:26 PM
F19814049: D9527.id22841.diff
Wed, Mar 4, 12:08 PM
F19813129: D9527.diff
Wed, Mar 4, 7:42 AM
F19811544: D9527.id22841.diff
Wed, Mar 4, 1:34 AM
F19806483: D9527.id.diff
Tue, Mar 3, 12:39 AM
F19802468: D9527.id.diff
Sun, Mar 1, 4:52 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).