@bgruening This is a follow up to PR #28. I agree that there were multiple places where I was unnecessarily using a regular expression when a simple string substitution is sufficient. However, the string method replace does not accept regular expressions (at least as is currently implemented in this script). Thus + file LICENSE is no longer removed, and the blank lines between sections are not added. This PR converts these two substitutions back to re.sub.
bgruening
commented
5 years ago
@bgruening This is a follow up to PR #28. I agree that there were multiple places where I was unnecessarily using a regular expression when a simple string substitution is sufficient. However, the string method
replacedoes not accept regular expressions (at least as is currently implemented in this script). Thus+ file LICENSEis no longer removed, and the blank lines between sections are not added. This PR converts these two substitutions back tore.sub.