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.

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

Already on GitHub? Sign in to your account

Small changes to compatibility for .NET #2404

Merged
merged 3 commits into from
Jan 4, 2023

Conversation

papafe
Copy link
Collaborator

@papafe papafe commented Dec 14, 2022

I did some small changes:

  • Made explicit we support .NET 5.0 and older versions. Strictly speaking .NET Core 2.0 and later includes .NET 5.0 but because the naming on .NET is very confusing, I think it makes sense to make it clear
  • Ordered the top part, so that iOS and MacOS sections are close, same for iOS and Android+
  • Removed the word "framework". The thing is that there is already .NET Framework, so the word framework here could be confusing... I could be convinced it makes sense though 馃榿
  • Modified a little the windows section so it seems slightly more consistent

@github-actions
Copy link

github-actions bot commented Dec 14, 2022

Readability for Commit Hash: 550765f

You can see any previous Readability scores (if they exist) by looking
at the comment's history.

Flesch Reading Ease scores for changed documents:

  • source/sdk/dotnet/compatibility: 61.63

The following table can be helpful in assessing the readability score of a document.

Score Difficulty
90-100 Very Easy
80-89 Easy
70-79 Fairly Easy
60-69 Medium
50-59 Fairly Hard
30-49 Hard
0-29 Very Hard

@MongoCaleb
Copy link
Collaborator

@papafe can you please rebase from upstream/master? That should fix the build errors.

@papafe
Copy link
Collaborator Author

papafe commented Jan 4, 2023

@MongoCaleb Sure, I've just done it

@MongoCaleb MongoCaleb merged commit 757ca4e into mongodb:master Jan 4, 2023
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.

None yet

2 participants