
<html><head></head><body>It's a nice detail, and doesn't hurt. +1<br><br><div class="gmail_quote">On March 23, 2019 7:51:30 AM GMT-03:00, Artur Juraszek <artur@juraszek.xyz> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">So, as far as I know treating the first line of commit messages<br>as _title_ and what follows as _body_ is a generally followed git practice,<br>and titles usually don't end with dots.<br><br>I realize it's a matter of personal preference to some extent though,<br>yet (according to me at least) they make the whole thing less readable -<br>similarly to using past tense, so I'd like to propose getting rid of them.<br><br>Signed-off-by: Artur Juraszek <artur@juraszek.xyz><hr>src/password-store.sh | 26 +++++++++++++-------------<br>1 file changed, 13 insertions(+), 13 deletions(-)<br><br>diff --git a/src/password-store.sh b/src/password-store.sh<br>index 284eabf..e7c5aa2 100755<br>--- a/src/password-store.sh<br>+++ b/src/password-store.sh<br>@@ -337,7 +337,7 @@ cmd_init() {<br>rm -v -f "$gpg_id" || exit 1<br>if [[ -n $INNER_GIT_DIR ]]; then<br>git -C "$INNER_GIT_DIR" rm -qr "$gpg_id"<br>- git_commit "Deinitialize ${gpg_id}${id_path:+ ($id_path)}."<br>+ git_commit "Deinitialize ${gpg_id}${id_path:+ ($id_path)}"<br>fi<br>rmdir -p "${gpg_id%/*}" 2>/dev/null<br>else<br>@@ -345,7 +345,7 @@ cmd_init() {<br>printf "%s\n" "$@" > "$gpg_id"<br>local id_print="$(printf "%s, " "$@")"<br>echo "Password store initialized for ${id_print%, }${id_path:+ ($id_path)}"<br>- git_add_file "$gpg_id" "Set GPG id to ${id_print%, }${id_path:+ ($id_path)}."<br>+ git_add_file "$gpg_id" "Set GPG id to ${id_print%, }${id_path:+ ($id_path)}"<br>if [[ -n $PASSWORD_STORE_SIGNING_KEY ]]; then<br>local signing_keys=( ) key<br>for key in $PASSWORD_STORE_SIGNING_KEY; do<br>@@ -354,12 +354,12 @@ cmd_init() {<br>$GPG "${GPG_OPTS[@]}" "${signing_keys[@]}" --detach-sign "$gpg_id" || die "Could not sign .gpg_id."<br>key="$($GPG --verify --status-fd=1 "$gpg_id.sig" "$gpg_id" 2>/dev/null | sed -n 's/^\[GNUPG:\] VALIDSIG [A-F0-9]\{40\} .* \([A-F0-9]\{40\}\)$/\1/p')"<br>[[ -n $key ]] || die "Signing of .gpg_id unsuccessful."<br>- git_add_file "$gpg_id.sig" "Signing new GPG id with ${key//[$IFS]/,}."<br>+ git_add_file "$gpg_id.sig" "Signing new GPG id with ${key//[$IFS]/,}"<br>fi<br>fi<br>reencrypt_path "$PREFIX/$id_path"<br>- git_add_file "$PREFIX/$id_path" "Reencrypt password store using new GPG id ${id_print%, }${id_path:+ ($id_path)}."<br>+ git_add_file "$PREFIX/$id_path" "Reencrypt password store using new GPG id ${id_print%, }${id_path:+ ($id_path)}"<br>}<br>cmd_show() {<br>@@ -476,7 +476,7 @@ cmd_insert() {<br>read -r -p "Enter password for $path: " -e password<br>echo "$password" | $GPG -e "${GPG_RECIPIENT_ARGS[@]}" -o "$passfile" "${GPG_OPTS[@]}" || die "Password encryption aborted."<br>fi<br>- git_add_file "$passfile" "Add given password for $path to store."<br>+ git_add_file "$passfile" "Add given password for $path to store"<br>}<br>cmd_edit() {<br>@@ -503,7 +503,7 @@ cmd_edit() {<br>while ! $GPG -e "${GPG_RECIPIENT_ARGS[@]}" -o "$passfile" "${GPG_OPTS[@]}" "$tmp_file"; do<br>yesno "GPG encryption failed. Would you like to try again?"<br>done<br>- git_add_file "$passfile" "$action password for $path using ${EDITOR:-vi}."<br>+ git_add_file "$passfile" "$action password for $path using ${EDITOR:-vi}"<br>}<br>cmd_generate() {<br>@@ -548,7 +548,7 @@ cmd_generate() {<br>fi<br>local verb="Add"<br>[[ $inplace -eq 1 ]] && verb="Replace"<br>- git_add_file "$passfile" "$verb generated password for ${path}."<br>+ git_add_file "$passfile" "$verb generated password for ${path}"<br>if [[ $clip -eq 1 ]]; then<br>clip "$pass" "$path"<br>@@ -586,7 +586,7 @@ cmd_delete() {<br>if [[ -n $INNER_GIT_DIR && ! -e $passfile ]]; then<br>git -C "$INNER_GIT_DIR" rm -qr "$passfile"<br>set_git "$passfile"<br>- git_commit "Remove $path from store."<br>+ git_commit "Remove $path from store"<br>fi<br>rmdir -p "${passfile%/*}" 2>/dev/null<br>}<br>@@ -630,19 +630,19 @@ cmd_copy_move() {<br>if [[ -n $INNER_GIT_DIR && ! -e $old_path ]]; then<br>git -C "$INNER_GIT_DIR" rm -qr "$old_path" 2>/dev/null<br>set_git "$new_path"<br>- git_add_file "$new_path" "Rename ${1} to ${2}."<br>+ git_add_file "$new_path" "Rename ${1} to ${2}"<br>fi<br>set_git "$old_path"<br>if [[ -n $INNER_GIT_DIR && ! -e $old_path ]]; then<br>git -C "$INNER_GIT_DIR" rm -qr "$old_path" 2>/dev/null<br>set_git "$old_path"<br>- [[ -n $(git -C "$INNER_GIT_DIR" status --porcelain "$old_path") ]] && git_commit "Remove ${1}."<br>+ [[ -n $(git -C "$INNER_GIT_DIR" status --porcelain "$old_path") ]] && git_commit "Remove ${1}"<br>fi<br>rmdir -p "$old_dir" 2>/dev/null<br>else<br>cp $interactive -r -v "$old_path" "$new_path" || exit 1<br>[[ -e "$new_path" ]] && reencrypt_path "$new_path"<br>- git_add_file "$new_path" "Copy ${1} to ${2}."<br>+ git_add_file "$new_path" "Copy ${1} to ${2}"<br>fi<br>}<br>@@ -651,10 +651,10 @@ cmd_git() {<br>if [[ $1 == "init" ]]; then<br>INNER_GIT_DIR="$PREFIX"<br>git -C "$INNER_GIT_DIR" "$@" || exit 1<br>- git_add_file "$PREFIX" "Add current contents of password store."<br>+ git_add_file "$PREFIX" "Add current contents of password store"<br>echo '*.gpg diff=gpg' > "$PREFIX/.gitattributes"<br>- git_add_file .gitattributes "Configure git repository for gpg file diff."<br>+ git_add_file .gitattributes "Configure git repository for gpg file diff"<br>git -C "$INNER_GIT_DIR" config --local diff.gpg.binary true<br>git -C "$INNER_GIT_DIR" config --local diff.gpg.textconv "$GPG -d ${GPG_OPTS[*]}"<br>elif [[ -n $INNER_GIT_DIR ]]; then<br></pre></blockquote></div><br>-- <br>HacKan || Iván</body></html>