idaholab / moose

Multiphysics Object Oriented Simulation Environment
https://www.mooseframework.org
GNU Lesser General Public License v2.1
1.7k stars 1.04k forks source link

clang-format check failing #12232

Closed WilkAndy closed 5 years ago

WilkAndy commented 5 years ago

Rationale

My latest PR ( #12231 ) is quite simple and fails the clang-format check. If this my fault, please close this issue immediately.

Description

Here's the output of civet:

Loaded modules 'moose-dev-clang miniconda'
Date: Mon Oct  1 19:25:43 MDT 2018
Machine: mobuild25
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.4 LTS
Release:    16.04
Codename:   xenial
MOOSE Package: Package version: 71
Build Root: /opt/civet/build_0
Trigger: Pull request
Step: Clang format check (1)
MODULESHOME: /opt/moose/Modules/3.2.10
Currently Loaded Modulefiles:
  1) moose/.gcc-7.3.0
  2) moose/.clang-5.0.1
  3) moose/.mpich-3.2_clang-5.0.1
  4) moose/.petsc-3.8.3_mpich-3.2_clang-5.0.1-opt
  5) moose/.tbb44_20150728
  6) moose/.cppunit-1.12.1_clang-5.0.1
  7) moose-dev-clang
  8) miniconda
Base commit: git@github.com:idaholab/moose.git:devel:1a2065b577e9c5d4256a91cdfd9538518e709852
Head commit: git@github.com:WilkAndy/moose.git:porosity_doco_12230:10c207cb78f8345b011e159b881cfbbd76dca42b

BUILD_ROOT/moose/: git merge-base devel 10c207cb78f8345b011e159b881cfbbd76dca42b
BUILD_ROOT/moose/: git clang-format 71668be662d2731ae1c95822ce13410f3262d507 -- framework/include framework/src test/include test/src modules/chemical_reactions/include modules/combined/include modules/contact/include modules/fluid_properties/include modules/functional_expansion_tools/include modules/heat_conduction/include modules/level_set/include modules/misc/include modules/module_loader/include modules/navier_stokes/include modules/phase_field/include modules/porous_flow/include modules/rdg/include modules/richards/include modules/solid_mechanics/include modules/stochastic_tools/include modules/tensor_mechanics/include modules/xfem/include modules/chemical_reactions/src modules/combined/src modules/contact/src modules/fluid_properties/src modules/functional_expansion_tools/src modules/heat_conduction/src modules/level_set/src modules/misc/src modules/module_loader/src modules/navier_stokes/src modules/phase_field/src modules/porous_flow/src modules/rdg/src modules/richards/src modules/solid_mechanics/src modules/stochastic_tools/src modules/tensor_mechanics/src modules/xfem/src examples tutorials unit stork
`git diff-files --name-status ` returned 128
fatal: ambiguous argument '': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
ERROR: exiting with code 2

Impact

Fix auto-checking of PRs.

brianmoose commented 5 years ago

12212 added some symbolic links that seem to break the clang format check.

Rebasing on devel and repushing should fix it

WilkAndy commented 5 years ago

Thanks @brianmoose ,

a