-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prefix user-facing interfaces with "I" to match .NET Convention #652
Comments
If this is to be done, it should be done before v1? Right? |
@HCanber yeah, we should just bite the bullet and do this now. Does anyone want this issue assigned to them? Should be straightforward. |
I'll try to take this one on. I haven't gotten to really contribute yet but with that in mind I'm happy to give it a shot. :) |
@Aaronontheweb Was this closed intentionally, or automatically by merging my PR? Because it isn't fixed. |
@HCanber whoops, I thought you said in your PR that this was finished. Looks like I misread it. My bad! |
These are the ones @nvivo listed in #633, and which have been fixed. Once this list is complete, we need to recheck and make sure there aren't any more.
|
I've done a few. Up for grabs for someone else. :) |
Just noting: once #801 is merged in, this list will be complete I think and can be closed. |
Closing; all interfaces listed in this issue have been accounted for and a new issue is opened for the additional items. |
Per #633, this should likely be undertaken after #651.
The text was updated successfully, but these errors were encountered: