StringExtensions confusing existing extensions and breaking build
See original GitHub issueI have utilized (among others) the string extensions StartsWithIgnoreCase
, EndsWithIgnoreCase
, and ContainsIgnoreCase
for use in many places in my organizations code.
Recently after updating this package I got a bunch of errors that it was ambiguous between the existing extensions and those found in the ShopifySharp.StringExtensions
class.
I would suggest to mark those internal as they are not critical to be exposed as part of this package and can, as I found out, actually cause compilation failures to downstream users.
Issue Analytics
- State:
- Created 4 years ago
- Comments:5 (3 by maintainers)
Top Results From Across the Web
c# - Debugging through an extension method
I landed here after having a problem getting VS to break in an extension method, I even restarted in case it was a...
Read more >When do you use extension methods? : r/csharp
I create extensions all the time, especially for extending functionality of an already existing class. For example, I have a ...
Read more >Proposal: extension everything · Issue #6136 · dotnet/roslyn
This is a good idea. It neatly solves the problem the design team has faced of how to add static extension methods with...
Read more >Champion "Extension function members" · Issue #192
I just encountered a scenario where I'm using an API and I'm unable to modify the base class but must use its derived...
Read more >Extension methods in C# | Exercises in .NET with Andras Nemes
Introduction Extension methods in C# allow you to extend the ... IEnumerable so the change in the signature doesn't break the existing code....
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
No problem! This has been published in 4.23.0 on Nuget. Thanks for the pull request!
Thanks! I’ll bump the version and publish it soon.