Skip to content

fix(cli): improve no stacks found message in synth output #3864

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nithishravindra
Copy link

@nithishravindra nithishravindra commented May 2, 2025

Related issue

Fixes #2793

Description

Updates the error message shown when no stacks are found during cdktf synth to be more descriptive. This helps users understand they need to define at least one TerraformStack in their CDKTF application

Checklist

  • I have updated the PR title to match CDKTF's style guide
  • I have run the linter on my code locally
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation if applicable
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works if applicable
  • New and existing unit tests pass locally with my changes

Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@nithishravindra nithishravindra force-pushed the improve-synth-no-stack-message branch from 39b12d9 to 61671c1 Compare May 16, 2025 16:47
@nithishravindra nithishravindra marked this pull request as ready for review May 16, 2025 16:52
@nithishravindra nithishravindra requested a review from a team as a code owner May 16, 2025 16:52
@nithishravindra nithishravindra requested review from ansgarm and DanielMSchmidt and removed request for a team May 16, 2025 16:52
@nithishravindra nithishravindra marked this pull request as draft May 16, 2025 16:53
@nithishravindra nithishravindra marked this pull request as ready for review May 16, 2025 17:01
@nithishravindra nithishravindra marked this pull request as draft May 16, 2025 17:06
@nithishravindra nithishravindra force-pushed the improve-synth-no-stack-message branch from c5e46ac to 1d6779f Compare May 20, 2025 05:16
@nithishravindra nithishravindra marked this pull request as ready for review May 20, 2025 05:23
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.

CLI UX: Nicer output for cdktf synth if there are no stacks in an app
1 participant