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

Already on GitHub? Sign in to your account

Use Array.Empty<T>() to efficiently create empty arrays #570

Merged
merged 2 commits into from
Oct 24, 2022

Conversation

deepsource-autofix[bot]
Copy link
Contributor

No description provided.

@hazzik hazzik enabled auto-merge (squash) October 24, 2022 21:23
@hazzik
Copy link
Member

hazzik commented Oct 24, 2022

Hi @pawan-deepsource, this auto-fix failed with

C:\projects\fluent-nhibernate\src\FluentNHibernate.Testing\Visitors\ComponentReferenceResolutionVisitorSpecs.cs(47,119): error CS0103: The name 'Array' does not exist in the current context [C:\projects\fluent-nhibernate\src\FluentNHibernate.Testing\FluentNHibernate.Testing.csproj]
C:\projects\fluent-nhibernate\src\FluentNHibernate.Testing\Visitors\ComponentReferenceResolutionVisitorSpecs.cs(47,119): error CS0103: The name 'Array' does not exist in the current context [C:\projects\fluent-nhibernate\src\FluentNHibernate.Testing\FluentNHibernate.Testing.csproj]

@hazzik hazzik merged commit 9b59d36 into master Oct 24, 2022
@hazzik hazzik deleted the deepsource-fix-254b3270 branch October 24, 2022 21:34
@pawan-deepsource
Copy link

pawan-deepsource commented Oct 24, 2022

Ah, looks like using System; wasn't inserted correctly. I'll stage and roll out a fix when I get into the office.

@hazzik hazzik added this to the vNext milestone Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants