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 macos build and support multiple files #81

Merged
merged 8 commits into from
Oct 21, 2023

Conversation

justin808
Copy link
Member

Conversion of template yaml to hash and back resulted in the loss of multiple documents in one yaml file.

MacOS builds require specification of the platform.

Conversion of template yaml to hash and back resulted in the loss of
multiple documents in one yaml file.

MacOS builds require specification of the platform.
lib/core/controlplane.rb Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated
# Run database migrations (or other release tasks) with latest image,
# while app is still running on previous image.
# This is analogous to the release phase.
cpl runner rails db:migrate -a tutorial-app --image latest
Copy link
Collaborator

Choose a reason for hiding this comment

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

We renamed the runner command to run:detached.

Copy link
Member Author

Choose a reason for hiding this comment

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

yes -- that needs to be fixed in the RWRT project as well @ahangarha

Copy link
Member Author

Choose a reason for hiding this comment

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

btw, should the command always be run detached? @dzirtusss ?

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@justin808
Copy link
Member Author

@rafaelgomesxyz ready to merge?

@rafaelgomesxyz
Copy link
Collaborator

@rafaelgomesxyz ready to merge?

From my side, yes.

@justin808 justin808 merged commit 267a488 into main Oct 21, 2023
9 checks passed
@justin808 justin808 deleted the fix-build-macos-run-multiple-docs-one-template branch October 21, 2023 03:27
@justin808
Copy link
Member Author

thank you @rafaelgomesxyz!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants