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

Migrate to new GH action #100

Merged
merged 2 commits into from
Jul 21, 2024
Merged

Migrate to new GH action #100

merged 2 commits into from
Jul 21, 2024

Conversation

0xTim
Copy link
Member

@0xTim 0xTim commented Jul 21, 2024

@0xTim 0xTim requested a review from gwynne as a code owner July 21, 2024 00:40
@0xTim 0xTim requested a review from ptoffy July 21, 2024 00:40
Copy link
Contributor

@MahdiBM MahdiBM left a comment

Choose a reason for hiding this comment

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

bucket-prefix: "vapor-blog-pull"
bucket-region: "us-east-1"
Copy link
Member

Choose a reason for hiding this comment

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

Isn't this eu-west-1?

Copy link
Member Author

Choose a reason for hiding this comment

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

The blog is actually in eu-west-2 but it doesn't matter where the PRs go, as long as it's the same between the open and close workflows

@0xTim 0xTim merged commit 0874f4c into main Jul 21, 2024
1 check passed
@0xTim 0xTim deleted the new-action branch July 21, 2024 16:27
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