Skip to content
This repository has been archived by the owner on Feb 28, 2022. It is now read-only.

Remove deprecated content.html property #176

Merged
merged 1 commit into from
Feb 11, 2019
Merged

Conversation

koraa
Copy link
Contributor

@koraa koraa commented Feb 5, 2019

The canonical way to get a string of the
html should be a call to content.document.body.innerHTML since content.html has been deprecated for a while.

The canonical way to get a string of the
html is to call content.document.body.innerHTML.
@codecov
Copy link

codecov bot commented Feb 6, 2019

Codecov Report

Merging #176 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #176   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          39     38    -1     
  Lines         833    828    -5     
  Branches      160    159    -1     
=====================================
- Hits          833    828    -5
Impacted Files Coverage Δ
src/defaults/html.pipe.js 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ca1499...d647575. Read the comment docs.

@koraa koraa requested review from tripodsan and trieloff February 6, 2019 15:06
@trieloff trieloff merged commit 7ee053f into master Feb 11, 2019
@koraa koraa deleted the karo/remove-html branch April 12, 2019 16:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants