Skip to content

Conversation

@slackapi
Copy link
Contributor

@slackapi slackapi bot commented Apr 10, 2025

This PR adds the docs!!!

@slackapi slackapi bot requested a review from a team as a code owner April 10, 2025 18:49
@slackapi slackapi bot added the docs M-T: Documentation work only label Apr 10, 2025
@codecov
Copy link

codecov bot commented Apr 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.94%. Comparing base (c335c36) to head (9205b05).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   62.94%   62.94%           
=======================================
  Files         210      210           
  Lines       22127    22127           
=======================================
  Hits        13928    13928           
  Misses       7115     7115           
  Partials     1084     1084           

☔ View full report in Codecov by Sentry.
📢 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.

@lukegalbraithrussell lukegalbraithrussell changed the title docs: open sources slack cli docs: open sources slack cli docs Apr 10, 2025
@mwbrooks mwbrooks added changelog Use on updates to be included in the release notes semver:patch Use on pull requests to describe the release version increment labels Apr 10, 2025
@mwbrooks mwbrooks added this to the Next Release milestone Apr 10, 2025
@mwbrooks mwbrooks changed the title docs: open sources slack cli docs docs: open source Slack CLI docs Apr 10, 2025
Copy link
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

✅ LGTM! Woo Woo! Docs are incoming! 📚 🚀 🔥

Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

👏 Having these docs included with the repo is so exciting for future changes all about. Thank so for bringing these files over.

I left a note on something small noticed, but no blocker at all 🚢 💨

Copy link
Member

Choose a reason for hiding this comment

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

📝 We might want to investigate the docgen command soon since I think this info is appearing on multiple pages:

But this is not a blocker or change from this PR!

Copy link
Member

Choose a reason for hiding this comment

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

🪓 #27

@lukegalbraithrussell lukegalbraithrussell merged commit c66672d into main Apr 10, 2025
6 checks passed
@lukegalbraithrussell lukegalbraithrussell deleted the docs-automated-sync-pr-109 branch April 10, 2025 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog Use on updates to be included in the release notes docs M-T: Documentation work only semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants