KEMBAR78
Scaffold IsFixedLength() instead of IsFixedLength(true) by KaloyanIT · Pull Request #23666 · dotnet/efcore · GitHub
Skip to content

Conversation

@KaloyanIT
Copy link
Contributor

#23473

  • Change annotation code generation for IsFixedLength()

Copy link
Member

@roji roji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good - see some minor corrections.

@roji roji changed the title Reverse Engineering generates not needed code Scaffold IsFixedLength() instead of IsFixedLength(true) Dec 12, 2020
@roji
Copy link
Member

roji commented Dec 13, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@roji roji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@smitpatel
Copy link
Contributor

/azp run efcore-ci (Build Windows)

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@roji roji merged commit 7e76a80 into dotnet:main Dec 15, 2020
@Saibamen
Copy link

Can this be shipped with 5.0.2?

@roji
Copy link
Member

roji commented Dec 15, 2020

@Saibamen as this is purely a cosmetic issue rather than a bug, there's little chance this could be considered for a 5.0 patch...

@Saibamen
Copy link

So there is nothing to wait for. This is just cosmetic issue, so there is no breaking changes

@roji
Copy link
Member

roji commented Dec 16, 2020

@Saibamen you probably want to take a look at the patching policy, specifically about the part where patches are about fixing bugs. Any change implies risk, so we don't merge cosmetics to patch releases.

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.

5 participants