Skip to content

Fix tests#22

Closed
panozzaj wants to merge 5 commits intophusion:masterfrom
panozzaj:fix-tests
Closed

Fix tests#22
panozzaj wants to merge 5 commits intophusion:masterfrom
panozzaj:fix-tests

Conversation

@panozzaj
Copy link
Copy Markdown
Contributor

Hi, we've been using this project for some time and it has served us well. Thank you for creating it!

Background: I'm trying to convert this gem's HTTP stack to use Faraday as an experiment / learning opportunity (and to potentially be able to take advantage of Faraday's middleware for retries since we've been getting a decent amount of 502/504 errors from Front).

As a first step, wanted to make sure that I could run the test suite. However, after getting on Ruby 3.1.2, I saw a few errors/failures as I ran rspec.

This PR just makes the test suite green (at least on my computer). Would this be helpful for this project? Thanks!

@panozzaj
Copy link
Copy Markdown
Contributor Author

This appears to be obsolete since I ran with Ruby 3.2.4 and got a single (different) test failure on the master branch. So I will close this PR.

@panozzaj panozzaj closed this Feb 13, 2025
@panozzaj
Copy link
Copy Markdown
Contributor Author

Also for any folks interested in converting to use Faraday, it looks like there is a fork that has some changes around that: master...stepful:frontapp:master

@panozzaj panozzaj deleted the fix-tests branch February 13, 2025 16:11
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