Skip to content

fix: remove the slack sync alias in favor of slack manifest sync - #674

Closed
zimeg wants to merge 1 commit into
mainfrom
remove-slack-sync-alias
Closed

fix: remove the slack sync alias in favor of slack manifest sync#674
zimeg wants to merge 1 commit into
mainfrom
remove-slack-sync-alias

Conversation

@zimeg

@zimeg zimeg commented Sep 11, 2026

Copy link
Copy Markdown
Member

Changelog

Removed the slack sync command alias. Use slack manifest sync instead.

Summary

This pull request removes the top-level slack sync alias in favor of slack manifest sync.

  • Drops the "sync" entry from AliasMap in cmd/root.go.
  • Regenerates command docs: removes the slack sync index entry and deletes the orphaned slack_sync.md reference page.
  • The slack manifest sync command itself is unchanged and remains the single way to sync a manifest between the local project and app settings.

Preview

N/A — CLI-only change.

Testing

  • Run ./bin/slack sync and confirm it reports unknown command "sync" for "slack".
  • Run ./bin/slack manifest sync --help and confirm the command still works as before.

Notes

slack manifest sync covers this workflow, so the shortcut alias is no longer needed.

Requirements

Drop the top-level slack sync shortcut from the alias map. The slack
manifest sync command remains the single way to sync a manifest between
the local project and app settings.
@zimeg
zimeg marked this pull request as ready for review September 11, 2026 21:13
@zimeg
zimeg requested review from a team as code owners September 11, 2026 21:13
@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.20%. Comparing base (fcb0782) to head (ed38f06).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #674      +/-   ##
==========================================
+ Coverage   78.19%   78.20%   +0.01%     
==========================================
  Files         239      239              
  Lines       18132    18132              
==========================================
+ Hits        14178    14181       +3     
+ Misses       3954     3951       -3     

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

@zimeg

zimeg commented Sep 11, 2026

Copy link
Copy Markdown
Member Author

🗣️ Conversations admist release hint that the shortcut is best so I close this change:

$ slack sync

@zimeg zimeg closed this Sep 11, 2026
@zimeg
zimeg deleted the remove-slack-sync-alias branch September 11, 2026 21:19
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