Skip to content

Deploy on S3 #45

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

Merged
merged 2 commits into from
Mar 5, 2025
Merged

Deploy on S3 #45

merged 2 commits into from
Mar 5, 2025

Conversation

StefanScherer
Copy link
Contributor

No description provided.

@StefanScherer
Copy link
Contributor Author

Thanks @nicksieger . I should have marked this PR as draft. We first need to test it, but I can't see the settings if GH actions are enabled, or what else is preventing the new workflow file from running. 🤔

@StefanScherer StefanScherer marked this pull request as draft March 4, 2025 16:02
@nicksieger
Copy link
Member

I don't think you can test it until it's merged? Maybe we set it up to be a workflow_dispatch first and trigger manually to test. Then if you need to make changes you can make them in a new PR and dispatch from that branch?

@StefanScherer
Copy link
Contributor Author

This other PR tilt-dev/tilt.build#1165 for the other websites works even before merging. That's why I was hoping. 😄

@nicks
Copy link
Member

nicks commented Mar 4, 2025

ya, i don't understand why the workflow didn't run

@nicks nicks force-pushed the gh_deploy_on_s3 branch 2 times, most recently from fd57063 to 1b9e817 Compare March 4, 2025 17:31
@nicks
Copy link
Member

nicks commented Mar 4, 2025

lol there was a typo

@nicks nicks force-pushed the gh_deploy_on_s3 branch 2 times, most recently from bc5ed02 to 0c3db44 Compare March 4, 2025 17:39
Signed-off-by: Stefan Scherer <[email protected]>
Signed-off-by: Nick Santos <[email protected]>
@nicks nicks force-pushed the gh_deploy_on_s3 branch from 0c3db44 to fc7d075 Compare March 4, 2025 17:46
@nicks
Copy link
Member

nicks commented Mar 4, 2025

wooo it works - https://api2.tilt.dev/

@nicks
Copy link
Member

nicks commented Mar 4, 2025

(i'll let @StefanScherer merge if there are any additional changes you wanted to make)

Signed-off-by: Stefan Scherer <[email protected]>
@StefanScherer StefanScherer marked this pull request as ready for review March 5, 2025 10:00
@StefanScherer StefanScherer merged commit 1a2abf6 into main Mar 5, 2025
5 checks passed
@StefanScherer StefanScherer deleted the gh_deploy_on_s3 branch March 5, 2025 10:00
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