Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Target net8.0 #11

Merged
merged 7 commits into from
Dec 24, 2023
Merged

Target net8.0 #11

merged 7 commits into from
Dec 24, 2023

Conversation

iamcarbon
Copy link
Contributor

This PR:

  • Updates the project target to net8.0
  • Updates xunit and fixes analyzer errors
  • Replaces LCP (longest common prefix) with optimized CommonPrefixLength
  • Replaces a few instances of HashSet with optimized SearchValues
  • Replaces a few readonly arrays with static data (this now works with primitive types)
  • Utilized the new ArgumentOutOfRangeException helpers

@iamcarbon
Copy link
Contributor Author

@PetroProtsyk Ready for review.

@PetroProtsyk PetroProtsyk merged commit 17bf39b into PetroProtsyk:master Dec 24, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@iamcarbon @PetroProtsyk