Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Problem: inconsistent formatting #2908

Merged
merged 7 commits into from
Feb 8, 2018
Merged

Problem: inconsistent formatting #2908

merged 7 commits into from
Feb 8, 2018

Commits on Feb 2, 2018

  1. Problem: no definition for applying automated formatting 

    Solution: added .clang-format file
    @sigiesec
    sigiesec committed Feb 2, 2018
    Configuration menu
    Copy the full SHA
    6d8baea View commit details
    Browse the repository at this point in the history

  2. Problem: formatting inconsistent 

    Solution: applied clang-format
    @sigiesec
    sigiesec committed Feb 2, 2018
    Configuration menu
    Copy the full SHA
    41f459e View commit details
    Browse the repository at this point in the history

  3. Problem: clang-format not run on CI 

    Solution: add clang-format to cmake, and add another travis-ci build type
    @sigiesec
    sigiesec committed Feb 2, 2018
    Configuration menu
    Copy the full SHA
    4161793 View commit details
    Browse the repository at this point in the history

  4. Problem: out-of-scope files are considered by cmake clang-format targets 

    Solution: restrict to src, tests, perf, tools and include directories
    @sigiesec
    sigiesec committed Feb 2, 2018
    Configuration menu
    Copy the full SHA
    26b5b5b View commit details
    Browse the repository at this point in the history

  5. Problem: travis-ci is using outdated clang(-format) 

    Solution: explicitly select clang-5.0
    @sigiesec
    sigiesec committed Feb 2, 2018
    Configuration menu
    Copy the full SHA
    749a85c View commit details
    Browse the repository at this point in the history

  6. Problem: clang-format-diff does not run 

    Solution: fix execution directory
    @sigiesec
    sigiesec committed Feb 2, 2018
    Configuration menu
    Copy the full SHA
    ac775ba View commit details
    Browse the repository at this point in the history

  7. Problem: cmake broken under Windows 

    Solution: fix CMAKE_MODULES_PATH
    @sigiesec
    sigiesec committed Feb 2, 2018
    Configuration menu
    Copy the full SHA
    43c11d0 View commit details
    Browse the repository at this point in the history