Skip to content

Comments

refactor: remove unused HTTParty dependency#19

Merged
billybonks merged 1 commit intomasterfrom
burn/httppary
Mar 8, 2025
Merged

refactor: remove unused HTTParty dependency#19
billybonks merged 1 commit intomasterfrom
burn/httppary

Conversation

@billybonks
Copy link
Contributor

No description provided.

Copy link

@managerbot-app managerbot-app bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏿 👎🏿 review comments so that we know what works and what does not work


# Specify your gem dependencies here
gem 'bundler', '~> 2.2'
gem 'httparty', '~> 0.20'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the HTTParty dependency indicates this component will no longer make HTTP requests directly. Ensure that any functionality that relied on HTTParty has been migrated to an alternative implementation.

@@ -1,6 +1,5 @@
# frozen_string_literal: true

require 'httparty'
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The HTTParty import is being removed, which suggests that commit_generator.rb no longer makes HTTP requests. Confirm that any HTTP functionality has been moved elsewhere or replaced with an alternative approach.

@billybonks billybonks merged commit f76dc57 into master Mar 8, 2025
2 checks passed
@billybonks billybonks deleted the burn/httppary branch March 8, 2025 10:46
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.

1 participant