[meta] Add .gitignore file in all repositories to enhance composer user usability about central HOT 17 CLOSED

Hello :-),

First, there is no link between Rush Release and Composer.
Then, maybe: vendor/ instead of /vendor/ would be better. What about vendor/*, what's the difference between them?

Also, should we add IDE/editors files, such as:

• *.swp for Vim,
• #… for Emacs,
• etc.

Do we assume users will use Vim, Emacs, PHPStorm and SublimeText to contribute? If yes, then we can use this gitignore file: https://www.gitignore.io/api/composer,vim,emacs,phpstorm,sublimetext.

Whilst gitignore.io seems to provide an exhaustive .gitignore file, what if we would like to add a new editor? We will have to patch all the .gitignore file. Is it an issue?

Thoughts? Should we git-ignore only Composer or also IDE and editors @hoaproject/hoackers?

from central.

Only ignore project specific stuff, such composer. Never clutter .gitignore with IDE stuff. For IDE files users MUST HAVE a gitignore global file. https://help.github.com/articles/ignoring-files/#create-a-global-gitignore

from central.

@patkar Agree with this.

from central.

@patkar +1

@Hywan

• vendor/ and vendor/* will produce the same result
• /vendor/ might be better as it will cause git to only ignore the vendor directory in the root of the repository. vendor/ will ignore it everywhere in the repo.

from central.

@jubianchi Thanks.
@shulard So… let's go with your proposal. Seems good for 👍!

from central.

@shulard What we used to do in this case is: Creating one issue per repository, with a link to this one. In this case, this issue will be a meta issue and will help to track the work. Then, you will have to create one PR per repository with a “Fix #xxx” in its description, where “xxx” is the ID of the issue (not the meta issue). We will close this meta issue manually when everything will be done.

from central.

@shulard Oh, and before creating PR for all repositories, let's try with one. Then we will review it, and after that, it will be the same work for all libraries 😃.

from central.

Hello,

Ok, I'll create the first PR. Maybe we should list all the repos that must be updated with that file.

from central.

@shulard Every library listed here: http://hoa-project.net/.

from central.

Oki 😄

from central.

I just updated the description of this issue to include the progression.

from central.

@shulard Check the checkboxes only when merged.

from central.

All the PRs have been created 😅

Sorry for the firsts, I mentioned the issue in the commit and in the PR...

from central.

Good work 👍!
I will try to merge everything after the lunch 😄!

from central.

Excellent work. Everything has been merged. Thank you very much 👍!

from central.

It just destroys the statistics of the week, haha:

from central.

The same for my contribution graph 😄

from central.