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

Removed usage of logger. #1760

Merged
merged 8 commits into from
Jul 17, 2019
Merged

Removed usage of logger. #1760

merged 8 commits into from
Jul 17, 2019

Conversation

ma2ciek
Copy link
Contributor

@ma2ciek ma2ciek commented Jul 10, 2019

Suggested merge commit message (convention)

Internal: Removed usage of the CKEditor 5 logger. Part of ckeditor/ckeditor5#383.

Additional information

For example – encountered issues, assumptions you had to make, other affected tickets, etc.

pjasiun
pjasiun reviewed Jul 12, 2019
View reviewed changes
src/view/view.js Outdated Show resolved Hide resolved
@pjasiun pjasiun requested a review from scofalik July 12, 2019 13:37
@coveralls
Copy link

coveralls commented Jul 17, 2019 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling f0278d2 on t/ckeditor5/383 into 9171150 on master.

@pjasiun pjasiun merged commit 0a268ab into master Jul 17, 2019
@pjasiun pjasiun deleted the t/ckeditor5/383 branch July 17, 2019 15:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@scofalik scofalik scofalik left review comments

@pjasiun pjasiun pjasiun left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ma2ciek @coveralls @pjasiun @scofalik