Replaced RestSharp with Refit
#25
Conversation
There was a problem hiding this comment.
Love the direction this is going, however there are quite a few failing tests.
Can you take a look? To run the tests, create a new organization at https://web.novu.co and grab the API key, and set the NOVU_API_KEY environment variable. If you use JetBrains Rider, you can set test environment variables by going to Settings > Unit Testing > Test Runner > Environment Variables at the bottom.
jetbrains://Rider/settings?name=Build%2C+Execution%2C+Deployment--Unit+Testing--Test+Runner
|
@wh1337 thx for review! sure will update tests prob tomorrow or on Monday |
|
@wh1337 I updated tests and code, so all seem good atm |
|
Don't worry about the failing action test. It's a known issue for PRs coming in via forks. I have ran the tests locally, and they have all passed:
@Badabum for your contribution! |
This pr replaces
RestSharplibrary withRefit.Refit is a more performant library and utilizes source generation to remove a lot of boilerplate code (as can already be seeing)
Related issue: #24