Skip to content

feat: add --manifest-source flag to run and deploy commands - #649

Draft
srtaalej wants to merge 1 commit into
mainfrom
ale-app-status-cmd
Draft

feat: add --manifest-source flag to run and deploy commands#649
srtaalej wants to merge 1 commit into
mainfrom
ale-app-status-cmd

Conversation

@srtaalej

Copy link
Copy Markdown
Contributor

Summary

  • Adds --manifest-source flag to slack run and slack deploy with values project or remote
  • Resolves manifest differences non-interactively using the specified source
  • Validates the flag value in PreRunE and provides clear error messaging
  • Updates non-TTY error remediation to reference --manifest-source instead of --force/--force-remote

Test plan

  • Verify --manifest-source=project resolves manifest diffs using local project
  • Verify --manifest-source=remote resolves manifest diffs using remote app settings
  • Verify invalid values produce clear error with remediation
  • Verify existing --force and --force-remote flags still work
  • Unit tests pass (make test)

@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.39%. Comparing base (b5982b9) to head (3eb0447).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
cmd/app/status.go 85.48% 5 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #649      +/-   ##
==========================================
+ Coverage   72.26%   72.39%   +0.12%     
==========================================
  Files         238      239       +1     
  Lines       20072    20135      +63     
==========================================
+ Hits        14505    14576      +71     
+ Misses       4298     4283      -15     
- Partials     1269     1276       +7     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@srtaalej
srtaalej force-pushed the ale-app-status-cmd branch from f153313 to 3eb0447 Compare August 27, 2026 16:27
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.

1 participant