Skip to content

[FSSDK-8918] chore: fix issues for prepare-release #495

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Apr 28, 2023
Merged

[FSSDK-8918] chore: fix issues for prepare-release #495

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
e54c35a
remove test_all from release script
jaeopt Apr 28, 2023
505ba49
add branch to pull-request
jaeopt Apr 28, 2023
f2a0c50
clean up
jaeopt Apr 28, 2023
6591c38
clean up
jaeopt Apr 28, 2023
a5aaa55
fix error
jaeopt Apr 28, 2023
abcbb98
fix hub command
jaeopt Apr 28, 2023
b19c32a
clean up
jaeopt Apr 28, 2023
6aaf754
clean up
jaeopt Apr 28, 2023
f0a8fd2
clean up
jaeopt Apr 28, 2023
b52c1b8
Merge branch 'master' into jae/fix-prep-2
jaeopt Apr 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 28 additions & 9 deletions Scripts/run_prep.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,13 +45,21 @@ function do_stuff {

# we need pod install or test_all.sh fails

# cocoapods requires ENV['HOME'] with absolute path
HOME=$(pwd)
gem install cocoapods -v $COCOAPODS_VERSION
pod _${COCOAPODS_VERSION}_ repo update
pod _${COCOAPODS_VERSION}_ install
# we skip "test_all.sh" until we have a good reason to repeat it heere
# 1. this test takes long and also flaky tests can interrupt release process
# 2. this "test_all.sh" is supposed to pass before starting pre-release
# 3. prep auto PRs will be tested in CI/CD before starting release process.

# - cocoapods requires ENV['HOME'] with absolute path
#
# HOME=$(pwd)
# gem install cocoapods -v $COCOAPODS_VERSION
# pod _${COCOAPODS_VERSION}_ repo update
# pod _${COCOAPODS_VERSION}_ install
#
# myscripts=( "update_version.sh ${VERSION}" "build_all.sh" "test_all.sh" )
myscripts=( "update_version.sh ${VERSION}" "build_all.sh" )

myscripts=( "update_version.sh ${VERSION}" "build_all.sh" "test_all.sh" )
for i in "${myscripts[@]}"; do
echo -n "${i} "
echo "===== ${i} =====" >> $BUILD_OUTPUT
Expand All @@ -67,8 +75,18 @@ function push_changes {
git config user.email "[email protected]"
git config user.name "${GITHUB_USER}"
git add --all
# this is like a try/catch
git commit -m "ci(git-action): auto release prep for $VERSION" ||

TITLE="ci(git-action): auto release prep for $VERSION"
# an empty line required between title and description
# a dummy ref (FSSDK-1234) for required FSSDK checking
MESSAGE=$(cat <<END
${TITLE}

- [FSSDK-1234]
END
)

git commit -m "${TITLE}" ||
{
case $? in
1 )
Expand All @@ -82,7 +100,8 @@ function push_changes {
esac
}
git push -f https://${GITHUB_TOKEN}@github.com/${REPO_SLUG} ${AUTOBRANCH}
PR_URL=$(hub pull-request --no-edit -b ${BRANCH})

PR_URL=$(hub pull-request -b ${BRANCH} -h ${AUTOBRANCH} -m "${MESSAGE}")
echo -e "${COLOR_CYAN}ATTENTION:${COLOR_RESET} review and merge ${COLOR_CYAN}${PR_URL}${COLOR_RESET}"
echo "then to release to cocoapods use Git action's Trigger build with the following payload:"
echo -e "${COLOR_MAGENTA}env:${COLOR_RESET}"
Expand Down