Multiple commits about drone-tree-config HOT 5 CLOSED

Drone does not always provide a last commit. Was the branch created with the push?

When drone provides no last build, drone-tree-config asumes just the last one. If you open a pull request the tests should be fine.

from drone-tree-config.

Yes, the branch was created with the push after the last commit. And you are right, pull request & Merge builds do solve the problem but we do some test on the branch level before pushing to master.

Maybe if I change this part to take the master branch as the before variable it would work? Or you think it will break something else?

// use diff to get changed files before := req.Build.Before if before == "0000000000000000000000000000000000000000" || before == "" { before = fmt.Sprintf("%s~1", req.Build.After) } var err error changedFiles, err = req.Client.ChangedFilesInDiff(ctx, before, req.Build.After)

from drone-tree-config.

i do not think its save to assume that the default branch is master.

also as far as i remember there where at least occasions where this happened

• new branches
• build restarts

currently i do not have the time for a full test setup. can you investigate further?

from drone-tree-config.

Hey, I found the bug and fixed it for our use case, i also edit the tests module, added the values needed for my solution to work, I opened a PR for this changes.

The problem was for builds triggered from push event, there was a git-diff between the last commit and the one before, that is wrong - when you push to remote branch in a multi branch environment - sometimes you have more than 1 commit.

scm.go:
} else { // use diff to get changed files before := req.Build.Before if before == "0000000000000000000000000000000000000000" || before == "" { before = fmt.Sprintf("%s~1", req.Build.After) }

There are 2 cases -

1. New branch - condition is true, the "before" variable is populated with the last commit with a suffix of "~1", (which i don't understand why you inserted the suffix), and the "after" variable is also the last commit.
2. Existing branch - condition is false, the "before" variable is populated with the commit before the push event and the "after" variable is also the last commit.

from drone-tree-config.

PR has been merged.

from drone-tree-config.