This function used to take an Option<&str> as commit message and provided a default. However, there was only one place where we passed None, so it's cleaner to just always pass the message.
We used to trim the message and append a new line at the end. However, git2 provides a function to clean up a commit message, which is used here now.
Test Plan: cargo build and use the fresh build to submit this Pull Request
This function used to take an
Option<&str>as commit message and provided a default. However, there was only one place where we passedNone, so it's cleaner to just always pass the message. We used to trim the message and append a new line at the end. However,git2provides a function to clean up a commit message, which is used here now.Test Plan:
cargo buildand use the fresh build to submit this Pull Request