Skip to content
Snippets Groups Projects

AnalysisBase: Add dummy flake8 configuration

Merged Frank Winklmeier requested to merge fwinkl/athena:anabase_flake8 into master
  1. Jun 09, 2020
    • Frank Winklmeier's avatar
      AnalysisBase: Add dummy flake8 configuration · 8181c17d
      Frank Winklmeier authored
      `flake8` is not available in AnalysisBase. While our cmake configuration
      does handle this correctly, it fails once developers start using special
      flake8 options, e.g.:
      ```
      ... POST_BUILD_CMD ${ATLAS_FLAKE8} --enable-extensions ATL902
      ```
      Even if `ATLAS_FLAKE8` is not defined it will now try to run a command
      of name `--enable-extensions`, which of course fails. An easy workaround
      is to `set( ATLAS_FLAKE8 true )` where `true` here is the shell builtin,
      which returns success and swallows any command line args given to it.
      8181c17d
    • Frank Winklmeier's avatar
      TrigBunchCrossingTool: Remove obsolete flake8 test · 1a6a6913
      Frank Winklmeier authored
      Remove obsolete flake8 test and the workaround for AnalysisBase.
      1a6a6913
Loading