slackapi / java-slack-sdk

Slack Developer Kit (including Bolt for Java) for any JVM language
https://slack.dev/java-slack-sdk/
MIT License
570 stars 209 forks source link

Make the same change to chat.postMessage with #1274 #1275

Closed seratch closed 7 months ago

seratch commented 7 months ago

This pull request applies the same change with #1274 to chat.postMessage API patterns. Actually, this is not a major problem for chat.postMessage at all because the API's rate limit is pretty generous. With that being said, having this change to it as well should improve the consistency of the SDK behavior.

Category (place an x in each of the [ ])

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (7c838c1) 74.49% compared to head (0a98fee) 74.53%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1275 +/- ## ============================================ + Coverage 74.49% 74.53% +0.04% - Complexity 4135 4137 +2 ============================================ Files 444 444 Lines 13108 13114 +6 Branches 1324 1325 +1 ============================================ + Hits 9765 9775 +10 + Misses 2570 2566 -4 Partials 773 773 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.