chore: deploy to AWS #2
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR sets up AWS infrastructure using Pulumi to deploy a static site with S3 and CloudFront, and adds a GitHub Actions workflow to automate deployments on pushes to the master branch.
- Configures S3 bucket, CloudFront distribution with Origin Access Control, optional ACM certificate provisioning, and Route53 DNS records
- Creates IAM role with OIDC provider for GitHub Actions authentication
- Adds deployment workflow that generates the static site and syncs to S3 with CloudFront cache invalidation
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| infra/index.ts | Core Pulumi infrastructure code defining S3, CloudFront, ACM, Route53, and IAM resources |
| infra/package.json | Project dependencies including Pulumi and AWS SDK packages |
| infra/tsconfig.json | TypeScript compiler configuration for the infrastructure code |
| infra/Pulumi.yaml | Pulumi project configuration specifying runtime and entry point |
| infra/Pulumi.dev.yaml | Development stack configuration with AWS region and project-specific settings |
| infra/.gitignore | Ignores Pulumi state directory from version control |
| infra/README.md | Documentation for infrastructure setup, deployment, and CloudFront logging configuration |
| infra/scripts/enable-cloudfront-logging.mjs | Script to configure CloudFront access logging with S3 bucket setup |
| .github/workflows/deploy.yml | GitHub Actions workflow for automated static site deployment |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Comment on lines
+28
to
+33
| pulumi config set githubOwner projectM-visualizer | ||
| pulumi config set githubRepo projectm-visualizer.org | ||
| pulumi config set githubRefs '["ref:refs/heads/master"]' | ||
| # Allow additional refs as needed, for example: | ||
| pulumi config set githubRefs '["ref:refs/heads/master","ref:refs/heads/*","ref:refs/tags/*","ref:refs/pull/*"]' | ||
| pulumi config set aws:region your-app-region |
There was a problem hiding this comment.
Inconsistent indentation in the code block. Lines 28-30 have extra leading spaces compared to line 27, which makes the commands appear misaligned. Ensure all lines have consistent indentation (no leading spaces after the newline).
Suggested change
| pulumi config set githubOwner projectM-visualizer | |
| pulumi config set githubRepo projectm-visualizer.org | |
| pulumi config set githubRefs '["ref:refs/heads/master"]' | |
| # Allow additional refs as needed, for example: | |
| pulumi config set githubRefs '["ref:refs/heads/master","ref:refs/heads/*","ref:refs/tags/*","ref:refs/pull/*"]' | |
| pulumi config set aws:region your-app-region | |
| pulumi config set githubOwner projectM-visualizer | |
| pulumi config set githubRepo projectm-visualizer.org | |
| pulumi config set githubRefs '["ref:refs/heads/master"]' | |
| # Allow additional refs as needed, for example: | |
| pulumi config set githubRefs '["ref:refs/heads/master","ref:refs/heads/*","ref:refs/tags/*","ref:refs/pull/*"]' | |
| pulumi config set aws:region your-app-region |
| pulumi config set githubRefs '["ref:refs/heads/master"]' | ||
| # Allow additional refs as needed, for example: | ||
| pulumi config set githubRefs '["ref:refs/heads/master","ref:refs/heads/*","ref:refs/tags/*","ref:refs/pull/*"]' | ||
| pulumi config set aws:region your-app-region |
There was a problem hiding this comment.
Line 33 has inconsistent indentation with extra leading spaces. This should align with the other commands in the code block to maintain consistency.
Suggested change
| pulumi config set aws:region your-app-region | |
| pulumi config set aws:region your-app-region |
blaquewithaq
approved these changes
Oct 25, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
AWS infra is configured with Pulumi (typescript that compiles to terraform)
Generates static site and pushes to the S3 bucket on push to master
Can configure DNS to point the domain at it