[pull] master from mattermost:master#809
Merged
Merged
Conversation
* Update docs-impact-review workflow with improved configuration - Include PR head SHA in concurrency group to avoid cancelling concurrent runs on different commits - Move permissions to workflow level - Remove Docs/Not Needed label skip condition from job if clause - Upgrade model to claude-sonnet-4-20250514 with increased max-turns (30→50) - Add Bash(ls*) to allowedTools - Add bug-fix pattern to no-docs-needed list - Refine security fix rule to skip docs unless externally observable contract change - Guard label addition behind !analysisFailed check Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Revert structural changes from #37084, keep prompt improvements Reverts the concurrency group SHA addition, permissions restructure, model/max-turns/tool changes, and label guard from #37084 as they are not needed. Keeps only the meaningful prompt changes: - Remove Docs/Not Needed label skip from job if condition - Add bug-fix pattern to no-docs-needed list - Refine security fix rule to skip docs unless externally observable contract change Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Restore Docs/Not Needed label skip condition Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Update docs-impact-review workflow --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
* chore: remove deprecated shared channels feature flags * chore: remove sharedChannelsPluginsEnabled prop from channel header * chore: remove enableSharedChannelsPlugins prop from file_search_results and post_message_view * chore: drop hardcoded FeatureFlagEnableSharedChannelsPlugins from client config * fix: remove unused channel prop from file_search_result_item * fix: remove double blank line in file_search_result_item test * remove redundant fragment wrapper in channel header * Remove unused channelId prop from CodeBlock
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )