fix: Fix duplicate printed lines when updating license header

[klmcadams]

In v0.7.1, the "Successfully changed {file}" lines are printed twice when a file is changed. This fixes that issue

[SMoraisAnsys]

Thanks for the fix @klmcadams. Could you help me understand what makes a line printed twice ? I tried to follow the flow in the code base and I'm stuck not understanding how this can happpen.

Since we only call non_recursive_file_check once, which loops on pre_commit_files, the if ... else ... statements should prevent us from facing that duplicate issue, no ?

[klmcadams]

Hi @SMoraisAnsys, no problem! This line add_header(copyright, license, years, file, template, commented, sys.stdout) wraps around reuse, and reuse prints whenever a header has been successfully changed. Then I have the if statement in following lines printing out the success message as well. So to get around it printing twice, I put the add_header output into a tmp file

[SMoraisAnsys]

Thanks for the explanation ! Here are a few comments, LGTM otherwise

[SMoraisAnsys]

Forgot to add my review comments xD

[SMoraisAnsys]

@klmcadams You need to add the import to io, I didn't add it on the review (sry)

[klmcadams]

@SMoraisAnsys, that's ok, no worries. I'll test it

[klmcadams]

@SMoraisAnsys, it looks like all tests passed so I'm going to merge it!