Skip to content

feat: make pull request body inclusion in Slack notifications optional - #6794

Open
olegglushak-megaport wants to merge 1 commit into
runatlantis:mainfrom
olegglushak-megaport:feat/slack-include-body-optional
Open

feat: make pull request body inclusion in Slack notifications optional#6794
olegglushak-megaport wants to merge 1 commit into
runatlantis:mainfrom
olegglushak-megaport:feat/slack-include-body-optional

Conversation

@olegglushak-megaport

@olegglushak-megaport olegglushak-megaport commented Aug 20, 2026

Copy link
Copy Markdown

This replaces #6633, whose author appears inactive, and addresses the review feedback left there.

@olegglushak-megaport
olegglushak-megaport marked this pull request as ready for review August 20, 2026 21:17
@dosubot dosubot Bot added feature New functionality/enhancement go Pull requests that update Go code labels Aug 20, 2026
@github-actions github-actions Bot added the docs Documentation label Aug 20, 2026
Add a --slack-include-body flag (default true, preserving current behavior) that lets operators drop the pull request body from Slack apply notifications. The setting threads through UserConfig into the Slack client and gates the Description attachment field on it.

Replaces runatlantis#6633 (author inactive), addressing the review there:
- set IncludeBody on the four zero-value DefaultSlackClient test constructions that assert the Description field (they broke as-is)
- add a test for the disabled path: body present, IncludeBody false, so no Description field
- order the flag alphabetically in the const block, UserConfig, test flags, and the server-configuration docs

Signed-off-by: Oleg Glushak <oleg.glushak@megaport.com>
@olegglushak-megaport
olegglushak-megaport force-pushed the feat/slack-include-body-optional branch from 6baf61f to 6d88ea8 Compare August 20, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation feature New functionality/enhancement go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant