Project

General

Custom queries

Profile

Actions

Optimization #7292

closed

CI: clang-format rechecks every main-7.0.x commit

Added by Philippe Antoine 9 months ago. Updated 23 days ago.

Status:
Closed
Priority:
Normal
Target version:
Effort:
Difficulty:
Label:

Description

This led to wrong formatting when fixing git conflict cf https://github.com/OISF/suricata/pull/11849


Subtasks 1 (0 open1 closed)

Bug #7293: CI: clang-format does not work for main-7.0.x branch (7.0.x backport)ClosedPhilippe AntoineActions
Actions #3

Updated by Philippe Antoine 9 months ago

  • Subject changed from CI: clang-format does not work for main-7.0.x branch to CI: clang-format rechecks every main-7.0.x commit

Because of FirstCommitOfBranch using master

Actions #5

Updated by Victor Julien 5 months ago

Why is this targeting 8?

Actions #6

Updated by Philippe Antoine 5 months ago

We need to fix it master first, otherwise, we will have the same problem when working on suricata 9 and doing backports for 8

Actions #8

Updated by Philippe Antoine about 1 month ago

function FirstCommitOfBranch {
    local first_commit=$(git rev-list origin/master..HEAD | tail -n 1)
    echo $first_commit
}

We need to use something else than origin/master. here

Actions #9

Updated by Philippe Antoine 26 days ago

  • Status changed from New to In Review
Actions #10

Updated by Philippe Antoine 25 days ago

  • Status changed from In Review to Resolved
Actions

Also available in: Atom PDF