Skip to content

feat: update version prefix to 2.1.0-preview and upgrade package depe…#37

Merged
ncipollina merged 1 commit into
mainfrom
feature/static-site-fix
Oct 15, 2025
Merged

feat: update version prefix to 2.1.0-preview and upgrade package depe…#37
ncipollina merged 1 commit into
mainfrom
feature/static-site-fix

Conversation

@ncipollina

@ncipollina ncipollina commented Oct 15, 2025

Copy link
Copy Markdown
Contributor

…ndencies

🚀 Pull Request

📋 Summary

Briefly describe what this PR does and why it's needed.


✅ Checklist

  • My changes build cleanly
  • I’ve added/updated relevant tests
  • I’ve added/updated documentation or README
  • I’ve followed the coding style for this project
  • I’ve tested the changes locally (if applicable)

🧪 Related Issues or PRs

Closes #...


💬 Notes for Reviewers

Any specific areas to look at, known issues, or follow-up work.

@ncipollina

Copy link
Copy Markdown
Contributor Author

@codex review

1 similar comment
@ncipollina

Copy link
Copy Markdown
Contributor Author

@codex review

@ncipollina
ncipollina requested a review from Copilot October 15, 2025 14:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the project version to 2.1.0-preview and upgrades various package dependencies. It also refactors the StaticSiteConstruct to use a distribution reference instead of the direct distribution object for CloudFront deployment.

  • Update version prefix from 2.0.1 to 2.1.0-preview
  • Upgrade CDK and testing framework dependencies
  • Refactor StaticSiteConstruct to use distribution reference pattern

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
Directory.Build.props Updates project version prefix to 2.1.0-preview
src/LayeredCraft.Cdk.Constructs/LayeredCraft.Cdk.Constructs.csproj Upgrades Amazon CDK and related package dependencies
test/LayeredCraft.Cdk.Constructs.Tests/LayeredCraft.Cdk.Constructs.Tests.csproj Updates testing framework and assertion library versions
src/LayeredCraft.Cdk.Constructs/StaticSiteConstruct.cs Refactors to use distribution reference for BucketDeployment

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/LayeredCraft.Cdk.Constructs/StaticSiteConstruct.cs
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

@ncipollina
ncipollina merged commit 3f1707b into main Oct 15, 2025
1 check passed
@ncipollina
ncipollina deleted the feature/static-site-fix branch October 15, 2025 14:36
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.

2 participants