Skip to content
Snippets Groups Projects

make it possible to use clang-tidy

Merged swym requested to merge swym/firmware:tidy into master
All threads resolved!

Contains a .jq script that mangles build/compile_commands.json in a way that is suitable for consumption by clang:

  • remove unknown command line options
  • add -m32 switch
  • make paths absolute (better for some IDEs)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • rahix
  • swym added 1 commit

    added 1 commit

    Compare with previous version

    • Resolved by rahix

      Two more things:

      • You should probably at least touch the @OUTPUT@ file, just in case
      • Please add compile-commands.json to .gitignore
  • swym added 1 commit

    added 1 commit

    Compare with previous version

  • rahix resolved all threads

    resolved all threads

  • swym resolved all threads

    resolved all threads

  • rahix assigned to @rahix

    assigned to @rahix

  • swym added 1 commit

    added 1 commit

    • 33efe1e9 - add compile_commands.json to .gitignore

    Compare with previous version

  • swym added 1 commit

    added 1 commit

    • 7664e314 - fix error thrown by jq sometimes - I don't know why, this just works better

    Compare with previous version

  • swym added 2 commits

    added 2 commits

    • 2d56a356 - touch @OUTPUT@
    • 70aa1f28 - give mangled compile_commands a more descriptive name

    Compare with previous version

  • rahix
  • swym resolved all threads

    resolved all threads

  • Thanks! Give me a sec, I'll try it out and merge

  • rahix added 2 commits

    added 2 commits

    • b806d383 - fix(build): Don't reference builddir in inputs
    • 7bf030ce - Fix jq script

    Compare with previous version

  • Looks good to me! Thanks for keeping our codebase clean.

    However, I'm only a bot - so a human will still have to approve this request.

  • added 3-Other label

  • rahix resolved all threads

    resolved all threads

  • merged

  • Please register or sign in to reply
    Loading