microsoft / microsoft/DacFx

IgnoreRespectNulls returns as List and should be a different type

Open
#181 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C#
Stars
460
Forks
29
Avg merge
4d 9h
Merged PRs (30d)
7

Description

We would expect a proper Fragment class IgnoreRespectNullsClause or maybe a simple a simple enum

Maybe something along the following lines?

public enum IgnoreRespectNulls
{
NotSpecified,
IgnoreNulls,
RespectNulls
}

Contributor guide

Open the contributing guide

Research direction

Start by locating the API member that exposes IgnoreRespectNulls and checking its callers and tests. Compare the existing List return type with the proposed IgnoreRespectNullsClause or enum, then determine the compatible public API change. Done means the return type represents the three stated states and affected tests or consumers pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp, sql
Domain
backend-api-design
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.