Skip to content

CS0436 \ConcurrentHashSet\src\ConcurrentHashSet\NullableAttributes.cs… #17

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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

huster-songtao
Copy link

… 中的类型“MaybeNullWhenAttribute”与“System.Runtime, Version=6.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a”中的导入类型“MaybeNullWhenAttribute”冲突。请使用 ConcurrentHashSet\src\ConcurrentHashSet\NullableAttributes.cs 中定义的类型。 ConcurrentHashSet (net6) D:\github\ConcurrentHashSet\src\ConcurrentHashSet\ConcurrentHashSet.cs

Microsoft.SourceLink.GitHub use 1.1.1 version.

Delete NullableAttributes.cs

Support netstandard2.1 and net6

use namespace System.Collections.Concurrent

forconz added 2 commits March 21, 2022 19:52
… 中的类型“MaybeNullWhenAttribute”与“System.Runtime, Version=6.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a”中的导入类型“MaybeNullWhenAttribute”冲突。请使用 ConcurrentHashSet\src\ConcurrentHashSet\NullableAttributes.cs 中定义的类型。 ConcurrentHashSet (net6) D:\github\ConcurrentHashSet\src\ConcurrentHashSet\ConcurrentHashSet.cs

Microsoft.SourceLink.GitHub use 1.1.1 version.

Delete NullableAttributes.cs

Support netstandard2.1 and net6

use namespace System.Collections.Concurrent
…tal:127. time elapsed 7 ms

ConcurrentHashSet: stopwords_en_nltk.txt load as parallel finished, Count:127, total:127. time elapsed 78 ms
ConcurrentBag: stopwords_en_nltk.txt load as parallel finished, Count:127, total:127. time elapsed 3 ms
@i3arnon
Copy link
Owner

i3arnon commented Mar 22, 2022

Hi, I'm not really sure what this PR is trying to achieve.
It seems it contains many things combined together and I would prefer to deal with each in a separate PR/issue so we can discuss each change.

Also,

Delete NullableAttributes.cs

The MaybeNullWhenAttribute makes the nullability conditional on the return value. Which is better than using T?.

Support netstandard2.1 and net6

Why? there's no reason to target netstandard 2.1 as this isn't using any 2.1 features. And net46 is not recommended as a target.

use namespace System.Collections.Concurrent

This is .NET's namespace, which can't be used as this is an OSS library outside of .NET.

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.

3 participants