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

Fix: Linting warnings #815

Merged
merged 24 commits into from
Sep 4, 2020
Merged

Fix: Linting warnings #815

merged 24 commits into from
Sep 4, 2020

Conversation

benhalverson
Copy link
Member

Description

Addresses a few linting warnings

  • Added dompurify as a dependency and used it around dangerouslySetInnerHTML.
  • Disabled linting warning about dangerouslySetInnerHTML
  • Fixed linting warning in article.test.tsx about not using the description.

Copy link
Contributor

@designMoreWeb designMoreWeb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@nschonni nschonni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Think this is good, but it needs a rebase

@benhalverson benhalverson merged commit c5d3987 into nodejs:master Sep 4, 2020
@benhalverson benhalverson deleted the linting-warnings branch September 4, 2020 21:30
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