mgmt icon indicating copy to clipboard operation
mgmt copied to clipboard

Published 20 hours ago • verified publisher icon purpleidea

test: Fix commit testing

Open evrardjp opened this issue 3 years ago • 2 comments

Currently, the testing of commits is in CI, and is CI dependent.

This is a problem if we switch to a new CI, or if someone wants to test locally.

This fixes it by making sure the test/test-commit-message can be triggered locally, and finds the new commits. As a side benefit, test/test-commit-message easier to understand: It compares the HEAD of the branch until reaching a common ancestor in origin, listing all the SHAs, and send those to the testing functions.

Closes: https://github.com/purpleidea/mgmt/issues/643

evrardjp avatar Mar 03 '21 11:03 evrardjp

Removed the BREAK stuff patch, which is failing (on purpose). This is good to go.

evrardjp avatar Mar 05 '21 07:03 evrardjp

Updated: Was broken because I was using spaces. Replaced spaces with tabs. Yes, I had to sell my soul for this patch to land. :laughing:

evrardjp avatar Mar 05 '21 11:03 evrardjp