mirror of
https://github.com/pi-hole/pi-hole.git
synced 2024-11-21 21:53:43 +00:00
Apply suggestions from code review
Co-authored-by: DL6ER <DL6ER@users.noreply.github.com> Signed-off-by: yubiuser <ckoenig@posteo.de>
This commit is contained in:
parent
a023195c4d
commit
ff91db7b9a
2 changed files with 2 additions and 2 deletions
|
@ -58,7 +58,7 @@ GitCheckUpdateAvail() {
|
||||||
# need @{0}
|
# need @{0}
|
||||||
LOCAL="$(git rev-parse "@{0}")"
|
LOCAL="$(git rev-parse "@{0}")"
|
||||||
|
|
||||||
# check if the local branch ef is a branch or a tag on remote repo
|
# check if the local branch ref is a branch or a tag on remote repo
|
||||||
if git show-ref -q --verify "refs/remotes/origin/$curBranch" 2>/dev/null; then
|
if git show-ref -q --verify "refs/remotes/origin/$curBranch" 2>/dev/null; then
|
||||||
ref_is_tag=false
|
ref_is_tag=false
|
||||||
elif git show-ref -q --verify "refs/tags/$curBranch" 2>/dev/null; then
|
elif git show-ref -q --verify "refs/tags/$curBranch" 2>/dev/null; then
|
||||||
|
|
|
@ -478,7 +478,7 @@ update_repo() {
|
||||||
# Remove any leading "heads/" as $curBranch is later used to check if local branch is remote branch or tag
|
# Remove any leading "heads/" as $curBranch is later used to check if local branch is remote branch or tag
|
||||||
curBranch=$(git rev-parse --abbrev-ref HEAD| sed "s/^heads\///g")
|
curBranch=$(git rev-parse --abbrev-ref HEAD| sed "s/^heads\///g")
|
||||||
|
|
||||||
# check if the local branch ef is a branch or a tag on remote repo
|
# check if the local branch ref is a branch or a tag on remote repo
|
||||||
if git show-ref -q --verify "refs/remotes/origin/$curBranch" 2>/dev/null; then
|
if git show-ref -q --verify "refs/remotes/origin/$curBranch" 2>/dev/null; then
|
||||||
ref_is_branch=true
|
ref_is_branch=true
|
||||||
elif git show-ref -q --verify "refs/tags/$curBranch" 2>/dev/null; then
|
elif git show-ref -q --verify "refs/tags/$curBranch" 2>/dev/null; then
|
||||||
|
|
Loading…
Reference in a new issue