Skip to content

Feat/MI 291 support resource prefix aspect#118

Merged
kai-nguyen-aligent merged 6 commits intomainfrom
feat/MI-291-support-resource-prefix-aspect
Mar 26, 2026
Merged

Feat/MI 291 support resource prefix aspect#118
kai-nguyen-aligent merged 6 commits intomainfrom
feat/MI-291-support-resource-prefix-aspect

Conversation

@kai-nguyen-aligent
Copy link
Copy Markdown
Contributor

Description of the proposed changes

  • Support resource prefix aspect.
  • Separate playground deployment and normal cdk commands. This ensure we match with how Aligent workflow works while provide a good dev experiment.
  • Couple of small improvements.

⚠️ Please make sure the changes adhere to the guidelines mentioned in the Aligent Contribution Guide

Notes to reviewers

🛈 When you've finished leaving feedback, please add a final comment to the PR tagging the author, letting them know that you have finished leaving feedback

aligent-phil
aligent-phil previously approved these changes Mar 26, 2026
@kai-nguyen-aligent kai-nguyen-aligent dismissed TheOrangePuff’s stale review March 26, 2026 00:25

The review has been addressed but Daniel is currently on leave so I dismiss this to move forward.

Copy link
Copy Markdown
Contributor

@nathaniel-cruz-aligent nathaniel-cruz-aligent left a comment

Choose a reason for hiding this comment

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

Just missing imports in the main.ts.template but it is not a major issue. Those imports can be easily added once template is generated.

Copy link
Copy Markdown
Contributor

@nathaniel-cruz-aligent nathaniel-cruz-aligent left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread packages/nx-cdk/src/generators/preset/files/application/bin/main.ts.template Outdated
Copy link
Copy Markdown

@aaronmedina-dev aaronmedina-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@kai-nguyen-aligent kai-nguyen-aligent merged commit 91d16b0 into main Mar 26, 2026
2 checks passed
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.

5 participants