Remove delete_after_merge functionality

[anishagg17]

Fixes: #212

removed code for attemptDelete and DeleteAfterMerge functionality

[palantirtech]

Thanks for your interest in palantir/bulldozer, @anishagg17! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

[asvoboda]

Hey @anishagg17, thanks for the PR.

Do you mind also removing the examples as well?

bulldozer/config/examples/standard.bulldozer.v1.yml

Line 14 in 0bfd948

delete_after_merge: true

https://github.com/palantir/bulldozer/blob/19a4b0800b00bab36aaf30366800f1016983efb5/README.md#bulldozeryml-specification

[anishagg17]

sure, @asvoboda I have removed it from some tests too and even from the Readme

[anishagg17]

Oh, I think that after changing the test config I need to make changes to the expected output also. Am I correct?

[asvoboda]

If you run ./godelw verify locally, you can see the same output from the test.

It looks like the head var L158 in merge.go is now unused, and the compiler is complaining.

[anishagg17]

It looks like the head var L158 in merge.go is now unused, and the compiler is complaining.

Yeah, replaced it with an _.

[anishagg17]

@asvoboda can we merge this one?