You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jclouds.apache.org by Jeremy Daggett <no...@github.com> on 2014/01/07 23:32:36 UTC

[jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Simple fix. Ensure that we ignore *.DS_STORE files.  I have had some interesting git status messages recently!
You can merge this Pull Request by running:

  git pull https://github.com/rackerlabs/jclouds-examples git-fix

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds-examples/pull/29

-- Commit Summary --

  * Updated .gitignore to ignore Mac OS specific files

-- File Changes --

    M .gitignore (1)

-- Patch Links --

https://github.com/jclouds/jclouds-examples/pull/29.patch
https://github.com/jclouds/jclouds-examples/pull/29.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29

Re: [jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Posted by Andrew Phillips <no...@github.com>.
Merged to [master](https://git-wip-us.apache.org/repos/asf?p=jclouds-examples.git;a=commit;h=15e8825ad8ad3a171f648aeb21ffc1761d2cf8bc).

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29#issuecomment-31865258

Re: [jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Posted by Andrew Phillips <no...@github.com>.
> @@ -16,3 +16,4 @@ rackspace/bin/
>  lein
>  project.clj
>  .lein-deps-sum
> +*.DS_STORE

Do we need `*.DS_STORE` here or would `.DS_STORE` be enough?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29/files#r8709950

Re: [jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Posted by Noorul Islam K M <no...@github.com>.
You can also achieve this by not adding to project .gitignore

https://help.github.com/articles/ignoring-files#global-gitignore

+1 though

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29#issuecomment-31802835

Re: [jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Posted by Andrew Phillips <no...@github.com>.
> @demobox Squashed and pushed, now crossing fingers! ;)

No need to cross fingers - it's the jclouds-examples repo...didn't see that. @jdaggett: intentional?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29#issuecomment-31820389

Re: [jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Posted by Jeremy Daggett <no...@github.com>.
@demobox Squashed and pushed, now crossing fingers! ;)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29#issuecomment-31802442

Re: [jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Posted by Jeremy Daggett <no...@github.com>.
> @@ -16,3 +16,4 @@ rackspace/bin/
>  lein
>  project.clj
>  .lein-deps-sum
> +*.DS_STORE

Sure, that should be sufficient. 

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29/files#r8710308

Re: [jclouds-examples] Updated .gitignore to ignore Mac OS specific files (#29)

Posted by Andrew Phillips <no...@github.com>.
Not sure why the PR builders didn't trigger on this one. Could you squash the two commits? That should hopefully kick them off. Otherwise, +1 from me - thanks, @jdaggett!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/29#issuecomment-31789684