-
Notifications
You must be signed in to change notification settings - Fork 23
Conversation
…gnore to ignore various Emacs temp files)
This has been running for many days on the pulls.wpt website. Assigning to @lukebjerring to help me get this merged. |
.gitignore
Outdated
@@ -99,3 +99,14 @@ config.txt | |||
|
|||
# Mac Finder | |||
.DS_Store | |||
|
|||
# mattl uses Emacs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
foolip uses Emacs too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW, when I once tried to add *~
to the .gitignore of a project where the maintainer wasn't an Emacs user, it was not so well received and I was pointed to something like https://stackoverflow.com/a/7335487
I actually haven't set that up myself, but that is a way to not being annoyed by this kind of thing in new repos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the correct answer is to set up a global gitignore
file (there are default paths, you don't actually need to set any config path) containing OS and editor-specific ignores (and GitHub's gitignores repo includes standard ones for both).
Travis is failing because there is no .travis.yml, so it can be safely ignored for now. |
I went to check the "Build only if .travis.yml is present" bit to make that problem go away. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -97,5 +97,3 @@ config.txt | |||
# Vagrant | |||
.vagrant | |||
|
|||
# Mac Finder | |||
.DS_Store |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! This didn't belong here anyway :)
In adding support for an external database hosts to the pulls.wpt site --
bocoup/infrastructure-web-platform#8 -- I need to extend the code to support non-localhost databases. This is a work-in-progress PR for that.
This change is