Skip to content
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.

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

fix(ci): update brew-bump.sh to update remote first #3438

Merged
merged 2 commits into from
May 21, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
27 changes: 20 additions & 7 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,18 @@
- [Changelog](#changelog)
- [Next Version](#next-version)
- [New Features](#new-features)
- [3.10.1](#3101)
- [Bug Fixes](#bug-fixes)
- [Documentation](#documentation)
- [Development](#development)
- [3.10.0](#3100)
- [New Features](#new-features-1)
- [3.10.1](#3101)
- [Bug Fixes](#bug-fixes-1)
- [Documentation](#documentation-1)
- [Development](#development-1)
- [3.10.0](#3100)
- [New Features](#new-features-1)
- [Bug Fixes](#bug-fixes-2)
- [Documentation](#documentation-2)
- [Development](#development-2)
- [Previous versions](#previous-versions)

<!-- END doctoc generated TOC please keep comment here to allow auto update -->
Expand Down Expand Up @@ -59,8 +62,20 @@ VS Code v1.56.1

### New Features

- feat: support `extraInitContainers` in helm chart values
- feat: change `extraContainers` to support templating in helm chart
- feat: support `extraInitContainers` in helm chart values #3393 @strowk
- feat: change `extraContainers` to support templating in helm chart #3393 @strowk

### Bug Fixes

- fix: use correct command to Open Folder on Welcome page #3437 @jsjoeio

### Documentation

- item

### Development

- fix(ci): update brew-bump.sh to update remote first #3438 @jsjoeio

## 3.10.1

Expand All @@ -71,7 +86,6 @@ VS Code v1.56.1
- fix: Check the logged user instead of $USER #3330 @videlanicolas
- fix: Fix broken node_modules.asar symlink in npm package #3355 @code-asher
- fix: Update cloud agent to fix version issue #3342 @oxy
- fix: use correct command to Open Folder on Welcome page #3437 @jsjoeio

### Documentation

Expand All @@ -80,7 +94,6 @@ VS Code v1.56.1
- docs(maintaining): add merge strategies section #3379 @jsjoeio
- refactor: move default PR template #3375 @jsjoeio
- docs(contributing): add commits section #3377 @jsjoeio
- docs(maintaining): add process for release managers #3360 @jsjoeio

### Development

Expand Down
27 changes: 27 additions & 0 deletions ci/steps/brew-bump.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,36 @@ main() {
cd "$(dirname "$0")/../.."
# Only sourcing this so we get access to $VERSION
source ./ci/lib.sh

# NOTE: we need to make sure cdrci/homebrew-core
# is up-to-date
# otherwise, brew bump-formula-pr will use an
# outdated base
echo "Cloning cdrci/homebrew-core"
git clone https://github.com/cdrci/homebrew-core.git

echo "Changing into homebrew-core directory"
cd homebrew-core && pwd

echo "Adding Homebrew/homebrew-core as $(upstream)"
git remote add upstream https://github.com/Homebrew/homebrew-core.git

echo "Fetching upstream commits..."
git fetch upstream

echo "Merging in latest changes"
git merge upstream/master

echo "Pushing changes to cdrci/homebrew-core fork on GitHub"
git push origin master

# Find the docs for bump-formula-pr here
# https://github.com/Homebrew/brew/blob/master/Library/Homebrew/dev-cmd/bump-formula-pr.rb#L18
brew bump-formula-pr --force --version="${VERSION}" code-server --no-browse --no-audit

# Clean up and remove homebrew-core
cd ..
rm -rf homebrew-core
}

main "$@"