Skip to content

Fix RuboCop lint failures in examples and chat client#5

Merged
sonnyt merged 2 commits into
mainfrom
copilot/fix-lint-job-failure
Jul 15, 2026
Merged

Fix RuboCop lint failures in examples and chat client#5
sonnyt merged 2 commits into
mainfrom
copilot/fix-lint-job-failure

Conversation

Copilot AI commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

The CI lint job was failing with 3 autocorrectable RuboCop offenses.

Changes

  • examples/unify_api.rb: Remove useless assignments to chat and git — these objects were constructed but never referenced; the script already uses unify.chat / unify.git via the client.unify facade
  • lib/bundleup/unify/chat.rb: Flatten multi-line unless block in #message to modifier form, consistent with the rest of the file
# before
unless response.success?
  raise "Failed to post chat/channels/#{encoded_channel_id}/message: #{response.status}"
end

# after
raise "Failed to post chat/channels/#{encoded_channel_id}/message: #{response.status}" unless response.success?

Copilot AI changed the title [WIP] Fix failing GitHub Actions job lint Fix RuboCop lint failures in examples and chat client Jul 15, 2026
Copilot AI requested a review from sonnyt July 15, 2026 12:45
@sonnyt
sonnyt marked this pull request as ready for review July 15, 2026 12:48
@sonnyt
sonnyt merged commit e359fde into main Jul 15, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants