Skip to content

Fixing the use of the binary formatter in net461 so all unit tests now pass.#2535

Merged
nickfloyd merged 1 commit intooctokit:mainfrom
JonruAlveus:FixBinarySerializerOnNet461
Aug 9, 2022
Merged

Fixing the use of the binary formatter in net461 so all unit tests now pass.#2535
nickfloyd merged 1 commit intooctokit:mainfrom
JonruAlveus:FixBinarySerializerOnNet461

Conversation

@JonruAlveus
Copy link
Contributor

@JonruAlveus JonruAlveus commented Aug 9, 2022

Also removed the need to have binary formatter tests run only in framework as it also works on netcoreapp3.1.
#2496

…w pass. Also removed the need to have it run only in framework as it also works on netcoreapp3.1
Copy link
Contributor

@nickfloyd nickfloyd left a comment

Choose a reason for hiding this comment

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

Nice! Solid contrib here @JonruAlveus, thank you ❤️. I love cleaning the campground kind of things!

@nickfloyd nickfloyd merged commit bc156af into octokit:main Aug 9, 2022
@nickfloyd
Copy link
Contributor

release_notes: Fixed the use of the binary formatter in net461 so all unit tests now pass. Also removed the need to have it run only in framework as it also works on netcoreapp3.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants