Repository navigation
docs: fix unclosed info admonition in authorizing-the-slack-cli - #604
Merged
Merged
Conversation
The `:::info` block about `--skip-update` / `SLACK_SKIP_UPDATE` being ignored by `slack upgrade` was missing the closing `]` on its title. The unclosed title bracket caused the MDX renderer to mangle later content on the page, surfacing a stray `authorize-cli}` in the preview. Co-Authored-By: Claude <[email protected]>
zimeg
added
bug
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #604 +/- ##
==========================================
- Coverage 71.68% 71.67% -0.02%
==========================================
Files 226 226
Lines 19176 19176
==========================================
- Hits 13747 13744 -3
- Misses 4220 4221 +1
- Partials 1209 1211 +2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
zimeg
marked this pull request as ready for review
June 30, 2026 22:59
mwbrooks
approved these changes
Jun 30, 2026
| When the CLI runs in a non-interactive environment, such as inside a CI/CD pipeline or when the output is piped, the update check runs without the auto-update confirmation prompt. The notification text may still appear in command output, however. You can suppress the check entirely with the `--skip-update` flag or `SLACK_SKIP_UPDATE` environment variable to keep the automation output clean. | ||
|
|
||
| :::info[The `--skip-update` flag and the `SLACK_SKIP_UPDATE` environment variable are intentionally ignored when running the `slack upgrade` command directly, since that command's purpose is to check for updates. | ||
| :::info[The `--skip-update` flag and the `SLACK_SKIP_UPDATE` environment variable are intentionally ignored when running the `slack upgrade` command directly, since that command's purpose is to check for updates.] |
Member
There was a problem hiding this comment.
Wow, great eye. I had to look twice before seeing the addition! +++
Member
|
Thanks for the fix up @zimeg! I'll merge this now so we don't lose track of it! 🙇🏻 |
Member
Author
|
@mwbrooks Immense thanks once again! The "auto" merge feature is awkward for me with also so much changes scattered about... Let's keep up the momentum! 🚢 💨 |
zimeg
removed
the
bug
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changelog
Fixed a malformed
:::infoadmonition on the Authorizing the Slack CLI guide that was breaking page rendering.Summary
This pull request fixes an unclosed
:::infoadmonition in the Authorizing the Slack CLI guide.--skip-update/SLACK_SKIP_UPDATEbeing ignored byslack upgradewas missing the closing]on its title.authorize-cli}in the rendered page.]closes the title and restores correct rendering.Preview
https://docs.slack.dev/tools/slack-cli/guides/authorizing-the-slack-cli/#version-updates
Testing
slack upgradeignoring--skip-update/SLACK_SKIP_UPDATErenders as a normal callout box.authorize-cli}text appears near the top of the page.Notes
Source file:
docs/guides/authorizing-the-slack-cli.md. One-character fix (adds the closing]).Requirements