Skip to content
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

Drop unused Copyable interface #18

Merged
merged 1 commit into from Mar 23, 2015
Merged

Drop unused Copyable interface #18

merged 1 commit into from Mar 23, 2015

Conversation

thiemowmde
Copy link
Contributor

This interface is not used anywhere in our core and Wikibase code bases ("used" in the sense of "type hinted against"). #16 finally made the interface obsolete.

@mariushoch
Copy link
Member

+1

@thiemowmde thiemowmde added this to the 2.0 milestone Feb 2, 2015
@mariushoch
Copy link
Member

Release notes?

@thiemowmde
Copy link
Contributor Author

I edited the release notes just a minute before you asked. ;-)

Note: I checked if it's possible to drop one of the other interfaces in this component too, but they are all used several times in our code base.

mariushoch added a commit that referenced this pull request Mar 23, 2015
Drop unused Copyable interface
@mariushoch mariushoch merged commit 239e772 into master Mar 23, 2015
@mariushoch mariushoch deleted the copyable branch March 23, 2015 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants