Skip to content

feat(create): cleanup app creation outputs when slack create -app passed#598

Open
srtaalej wants to merge 1 commit into
mainfrom
ale-create-app-outputs
Open

feat(create): cleanup app creation outputs when slack create -app passed#598
srtaalej wants to merge 1 commit into
mainfrom
ale-create-app-outputs

Conversation

@srtaalej

Copy link
Copy Markdown
Contributor

Changelog

Reduced verbose output when using create --app to link an existing app during project creation.

Summary

When --app is provided to slack create, the linking step previously printed multiple noisy sections (App Link header, App Manifest details, App footer, and "Added existing app" message). Now it only prints the single 🏠 App summary section showing team, App ID, Team ID, User ID, and Status.

Preview

Screenshot 2026-06-22 at 4 16 44 PM

Testing

  • make test
  • Manual test - ./bin/slack create my-test -t slack-samples/bolt-js-starter-template --app A123 --team T123 --environment local

Notes

Adresses comments on PR #565 - Improve the output experience for create when --app is provided.

Requirements

@srtaalej srtaalej added this to the Next Release milestone Jun 22, 2026
@srtaalej srtaalej self-assigned this Jun 22, 2026
@srtaalej srtaalej requested a review from a team as a code owner June 22, 2026 20:21
@srtaalej srtaalej added enhancement M-T: A feature request for new functionality semver:minor Use on pull requests to describe the release version increment labels Jun 22, 2026
@srtaalej srtaalej changed the title feat: cleanup app creation outputs when passed feat(create): cleanup app creation outputs when slack create -app passed Jun 22, 2026
@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.59%. Comparing base (f0d20ad) to head (a0f86d7).

Files with missing lines Patch % Lines
cmd/app/link.go 0.00% 19 Missing ⚠️
cmd/project/create.go 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #598      +/-   ##
==========================================
- Coverage   71.64%   71.59%   -0.06%     
==========================================
  Files         226      226              
  Lines       19176    19195      +19     
==========================================
+ Hits        13739    13742       +3     
- Misses       4222     4240      +18     
+ Partials     1215     1213       -2     

☔ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement M-T: A feature request for new functionality semver:minor Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant