ipa320 / care-o-bot

http://www.care-o-bot.de
Apache License 2.0
48 stars 41 forks source link

ipa320 default gitignore #35

Closed fmessmer closed 7 years ago

fmessmer commented 7 years ago

@ipa-mig @ipa-fmw @ipa-nhg @ipa-bnm How do you like this one? related to https://github.com/ipa320/cob_calibration_data/pull/118

I'm not sure about bin and lib as we sometimes have files in these folder which we don't want to be gitignored...e.g. cob_extern.... I think we could remove build, bin and lib as catkin does not create those folders in the source space anymore...

fmessmer commented 7 years ago

considered @ipa-mig's comments @ipa-fmw please provide feedback...

After approval, I will provide PRs to the relevant repositories

fmessmer commented 7 years ago

Here is a list of candidates for the default .gitignore:

bringup packages

msh packages

Did I forget something? Should any of the packages above NOT use the default .gitignore?

mgruhler commented 7 years ago

all navigation packages could use the new one.

I guess the rest should be fine as well. I'm not sure about msh, though.

mgruhler commented 7 years ago

Still waiting for @ipa-fmw's feedback, though?

fmessmer commented 7 years ago

Yes, I'd like to see approval of @ipa-fmw and @ipa-nhg for this before creating 30 PRs...

fmessmer commented 7 years ago

Well, I'll go on with this now...I have three approvals...

fmessmer commented 7 years ago

updated all repos from the list above!