Paul Irwin

Task.Run in this case should be avoided because the .Send() method is IO-bound rather than CPU-bound. Task.Run should only be used for CPU-bound tasks. For IO-bound code, you should go...

I did just that. https://github.com/andrewdavey/postal/pull/116

I am looking into this and I think it's time for a radical rewrite of Harvest.Net to support API v2 and .NET Core (#63), due to RestSharp not supporting .NET...

My initial prototype of this is here: https://github.com/paulirwin/harvest.net/tree/netcore-rewrite This supports .NET Standard 2.0, GETs for the Clients and Companies Harvest v2 APIs so far, and uses Refit to remove the...

@kfrancis I don't have the OAuth-related bits hooked up yet but I will soon. However you can also use a Personal Access Token instead of an OAuth token in v2.

Not like I would expect a big outrage on this in 2021, but technically going from .NET 4.5 to 4.5.2 is a breaking change. My vote would be for a...

@jehhynes Ah sorry, you're right, my eyes caught one of the NuGet packages going from net45 to net452 when scanning the diff.

I'll gladly take this one once #75 is confirmed good to go and merged in.

@lprichar Awesome, this is only possible now because RestSharp now supports .NET Standard 2.0. It didn't at the time that I started a rather ambitious rewrite. I'd still like to...

@lprichar Can you make a PR for this? I've successfully tested it and at least for my use it works well.