Skip to content

Simplify gitignore#888

Closed
ischoegl wants to merge 2 commits into
Cantera:mainfrom
ischoegl:fix-gitignore
Closed

Simplify gitignore#888
ischoegl wants to merge 2 commits into
Cantera:mainfrom
ischoegl:fix-gitignore

Conversation

@ischoegl
Copy link
Copy Markdown
Member

@ischoegl ischoegl commented Jun 30, 2020

Checklist

  • There is a clear use-case for this code change
  • The commit message has a short title & references relevant issues
  • Build passes (scons build & scons test) and unit tests address code coverage
  • The pull request is ready for review

If applicable, fill in the issue number this pull request is fixing

  • In Improve error message for missing phases key #880 renamed executables were excluded in the main .gitignore file (rather than test_problems/.gitignore)
  • Hidden .gitinore files in subfolders are hard to locate / easy to overlook
  • Merge test_problems/.gititnore (and others) into .gitignore
  • Use include overrides in addition to exclude directives to declutter

Changes result in a reduction of gitignore instructions by half

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 30, 2020

Codecov Report

Merging #888 into main will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #888   +/-   ##
=======================================
  Coverage   71.11%   71.11%           
=======================================
  Files         376      376           
  Lines       46201    46201           
=======================================
  Hits        32858    32858           
  Misses      13343    13343           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 07dac7b...3287b40. Read the comment docs.

@ischoegl ischoegl changed the title Fix gitignore Simplify gitignore Jun 30, 2020
@speth speth changed the base branch from master to main June 30, 2020 23:15
ischoegl added 2 commits July 8, 2020 17:21
- merge test_problems/.gitignore with main file
- use exclude/include lists to declutter
- Move all .gitignore instructions to a single file
- Simplify files via exclusion/inclusion syntax
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants