Skip to content

MI-314: Add remove-service generator for nx-cdk#149

Open
kai-nguyen-aligent wants to merge 10 commits intomainfrom
fix/MI-314-fix-error-when-removing-nx-cdk-service
Open

MI-314: Add remove-service generator for nx-cdk#149
kai-nguyen-aligent wants to merge 10 commits intomainfrom
fix/MI-314-fix-error-when-removing-nx-cdk-service

Conversation

@kai-nguyen-aligent
Copy link
Copy Markdown
Contributor

Description of the proposed changes

  • Added a new remove-service generator to the nx-cdk package that cleanly removes a service and all of its references (imports, stack instantiation, tsconfig references, workspace entries)
  • Extracted shared constants (SERVICES_FOLDER, MAIN_APPLICATION_NAME, SERVICES_SCOPE) into a dedicated constants module for reuse across generators
  • Added --access public flag to the nx release publish command in the CI publish workflow
  • Renamed dependabot commit-message prefixes from TEMPLATE-UPDATE to NX-CDK-TEMPLATE / NX-APPBUILDER-TEMPLATE
  • Updated nx-cdk README with documentation for both the new remove-service generator and additional service generator options

Notes to reviewers

  • 🛈 Toggl Code: MI-314: Code Review
  • 🛈 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

Copy link
Copy Markdown
Contributor

@aligent-lturner aligent-lturner left a comment

Choose a reason for hiding this comment

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

Just a question regarding the package downgrade

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