Skip to content

Consider removing <Nullable>annotations</Nullable> from the Bonsai Foundation template #26

@PathogenDavid

Description

@PathogenDavid

<Nullable>annotations</Nullable>

This property was only added because nearly all of the repositories that were upgraded with Prefect did not support nullable reference types.

However for new packages, this probably isn't a good default and we should encourage using them instead.

This file has its content ignored, so removing the <Nullable> here will not cause Prefect to want to remove it from existing repositories.

And it can just be removed from this file entirely as <Nullable> is already configured in Common.csproj.props.

(You could actually just remove Project.csproj.props from the template entirely as it's completely optional. That would hurt the ability for people to discover it as an extension point though.)

Metadata

Metadata

Assignees

Labels

area/reference-bonsaiIssues concerning the Bonsai Foundation reference templateaspect/best-practiceIssues concerning the proliferation of misc best practices not covered by other aspect tags.

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions