Skip to content

feat(dotnet): drop the useless I prefix for non datatype interfaces#728

Merged
RomainMuller merged 1 commit intomasterfrom
hamzaad/drop-I-interface
Aug 22, 2019
Merged

feat(dotnet): drop the useless I prefix for non datatype interfaces#728
RomainMuller merged 1 commit intomasterfrom
hamzaad/drop-I-interface

Conversation

@assyadh
Copy link
Copy Markdown
Contributor

@assyadh assyadh commented Aug 21, 2019

Drop the useless I prefix for non datatype interfaces as they are guaranteed by jsii to start with I.

This removes a bunch of II* interfaces.

Updated the tests, and tested with the CDK successfully.

BREAKING CHANGE: names of .NET behavioral interfaces have changed (the duplicate prefix I was removed).

Fixes #109


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@assyadh assyadh requested review from a team and costleya as code owners August 21, 2019 20:07
@assyadh assyadh assigned assyadh and unassigned assyadh Aug 21, 2019
@assyadh assyadh added the language/dotnet Related to .NET bindings (C#, F#, ...) label Aug 21, 2019
@assyadh assyadh requested a review from RomainMuller August 21, 2019 20:24
@RomainMuller RomainMuller merged commit b9621f1 into master Aug 22, 2019
@RomainMuller RomainMuller deleted the hamzaad/drop-I-interface branch August 22, 2019 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

language/dotnet Related to .NET bindings (C#, F#, ...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When a JSII interface has a name like IFoo, the generated .NET interface is named IIFoo

2 participants