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

Jump to bottom

Add build and release configuration for Upsun and other Vendors #127

Merged

akalipetis merged 7 commits into main from akalipetis/issue-110

Sep 28, 2023

Member

akalipetis commented Sep 26, 2023 •

edited

Loading

The Upsun CLI is built alongside the Platform.sh CLI and are distributed together.

For the rest of the vendors, a .goreleaser.vendor.yaml and an internal/config/embedded-config.yaml should be provided.

Fix #110

akalipetis requested review from lolautruche and pjcdawkins

September 26, 2023 14:42

akalipetis added 2 commits

September 26, 2023 17:46


          Add build and release configuration for Upsun and other Vendors

e7590ab

The Upsun CLI is built alongside the Platform.sh CLI and are distributed together.

For the rest of the vendors, a .goreleaser.vendor.yaml and an internal/config/embedded-config.yaml should be provided.


          Fix default config flavor to be platform

a67b508

akalipetis force-pushed the akalipetis/issue-110 branch from 149daf9 to a67b508 Compare

September 26, 2023 14:49

pjcdawkins reviewed

View reviewed changes

internal/config/schema.go Outdated Show resolved Hide resolved

akalipetis and others added 2 commits

September 26, 2023 20:00


          Vendorize the installer and Cloudsmith upload

797588f


          Update internal/config/schema.go

0929bb4

Co-authored-by: Patrick Dawkins <[email protected]>

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

internal/config/upsun-cli.yaml Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

upsun-staging.phar Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

installer.sh Outdated Show resolved Hide resolved

pjcdawkins reviewed

View reviewed changes

installer.sh Outdated Show resolved Hide resolved


          Fix review comments by @pjcdawkins

platformsh deleted a comment from pjcdawkins

akalipetis and others added 2 commits

September 28, 2023 14:23


          Apply suggestions from code review

ac826f6

Co-authored-by: Patrick Dawkins <[email protected]>


          Disable deprecated and invalid commands from the Upsun CLI

a0e5da8

pjcdawkins approved these changes

View reviewed changes

akalipetis merged commit f9f337d into main

akalipetis deleted the akalipetis/issue-110 branch

September 28, 2023 15:08

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet