Skip to content

Conversation

@CyberShadow
Copy link

No description provided.

@filipw
Copy link
Member

filipw commented Oct 19, 2021

thanks - unfortunately this typo is in the public interface which is published to nuget and it is a breaking change.
However IMHO this is still acceptable change to make, as it should have never been like this in the first place, it's a clear mistake.

@filipw
Copy link
Member

filipw commented Oct 21, 2021

@NTaylorMullen do you guys implement this interface in Razor tooling? would this be a breaking change if we rename this due to a typo?

@NTaylorMullen
Copy link

@filipw thanks for asking. So based on a search of our repo I don't see any reference of Initalize which leads me to think this would be fine. That being said, the assumption there assumes that O# is binding redirecting dlls appropriately so we don't typeload 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants