Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

T/1616: Hide wrap from public API #1625

Merged
merged 3 commits into from
Jan 2, 2019
Merged

T/1616: Hide wrap from public API #1625

merged 3 commits into from
Jan 2, 2019

Conversation

jodator
Copy link
Contributor

@jodator jodator commented Jan 2, 2019 •
edited by pomek
Loading

Suggested merge commit message (convention)

Other: Remove wrap from public API. Closes ckeditor/ckeditor5#4453.

BREAKING CHANGE: The wrap() conversion helper was removed from public API.

Additional information

@jodator jodator requested a review from pjasiun January 2, 2019 13:19
@coveralls
Copy link

coveralls commented Jan 2, 2019 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling b630e8d on t/1616 into 85d6f75 on master.

@jodator
Copy link
Contributor Author

jodator commented Jan 2, 2019

Tests fails because of #1624.

@pjasiun pjasiun merged commit 1c7ef68 into master Jan 2, 2019
@pjasiun pjasiun deleted the t/1616 branch January 2, 2019 14:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pjasiun pjasiun Awaiting requested review from pjasiun

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@jodator @coveralls @pjasiun