IgnoreRespectNulls returns as List and should be a different type
Open
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
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