Update .gitignore Required - Githubissues

cms-gem-daq-project / xhal

XHAL interface library

0 stars 10 forks source link

Update .gitignore Required #22

Closed bdorney closed 7 years ago

bdorney commented 7 years ago

Brief summary of issue

While one should not use git add . I think the .gitignore needs to be updated. For example it is not setup to ignore temporary files, e.g.:

Untracked files:
  (use "git add <file>..." to include in what will be committed)

    include/xhal/rpc/optohybrid.h
    src/common/rpc_manager/.vfat3.cc.swp

Types of issue

[ ] Bug report (report an issue with the code)
[X] Feature request (request for change which adds functionality)

Expected Behavior

Temporary files should not be known to git.

Additionally, I think it would be good to adopt a common .gitignore structure in all our repo's. Looking at:

cmsgemos .gitignore,
gem-plotting-tools .gitignore
vfatqc-python-scripts .gitignore

They seem to have the following structure:

# Ignore everything
*

# Don't ignore git settings
!*.gitignore
!*.gitattributes

# Don't ignore special files
!*.md
!*.rst

# Don't ignore subdirectories
!*/

# Don't ignore special files to this repo that don't fit in the above
<Files/reg expressions>

Current Behavior

Temporary files are known to git.