Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Select last commit or full branch history for analysis of #bumps #218

Merged
merged 3 commits into from
Dec 20, 2022

Conversation

sbe-arg
Copy link
Collaborator

@sbe-arg sbe-arg commented Dec 4, 2022

Variable BRANCH_HISTORY" full|last allows to select the length of the history for finding bump keywords.

As per examples:

history='full'
declare -A history_type=( 
    ["last"]="$(git show -s --format=%B)" \
    ["full"]="$(git log master..HEAD --format=%B)" \
)
log=${history_type[${history}]}
printf "History:\n---\n%s\n---\n" "$log"
History:
---
history type


POC log
---
history='last'
declare -A history_type=( 
    ["last"]="$(git show -s --format=%B)" \
    ["full"]="$(git log master..HEAD --format=%B)" \
)
log=${history_type[${history}]}
printf "History:\n---\n%s\n---\n" "$log"
History:
---
history type
---

fixes #211

README.md Show resolved Hide resolved
@sammcj
Copy link
Collaborator

sammcj commented Dec 4, 2022

Just that one tiny comment / question otherwise ok to approve!

@sammcj sammcj self-requested a review December 5, 2022 04:17
Copy link
Collaborator

@sammcj sammcj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@sbe-arg
Copy link
Collaborator Author

sbe-arg commented Dec 11, 2022

Been sick and afk will merge in the next few days.

@sbe-arg sbe-arg merged commit 2fa4983 into master Dec 20, 2022
@sbe-arg sbe-arg deleted the feature/flag-history-context branch December 20, 2022 01:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Breaking change introduced with 1.51.0 breaks tag workflow
2 participants