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

Removes duplicate version of the prism-react-renderer #1063

Merged
merged 1 commit into from
Jan 31, 2022

Conversation

JKarlavige
Copy link
Collaborator

@JKarlavige JKarlavige commented Jan 28, 2022

Description & motivation

Asana task: https://app.asana.com/0/1200099998847559/1201682203530869/f

Snyk flagged an outdated PrismJS package. This package is being used by the prism-react-renderer package, which is included in Docusaurus. While we are unable to edit the dependency used by the Docusaurus theme, I noticed the package in our package.json is unneeded, as it is already included in Docusaurus. This removes prism-react-renderer from our package.json, which looks like it was added when I initially upgraded Docusaurus.

Note, I do not expect this to correct the Snyk flag, as the outdated package is still used by Docusaurus. There is an open Git issue here, and noted by Docusaurus here.

Pre-release docs

Is this change related to an unreleased version of dbt?

  • Yes: please
    • update the base branch to next
    • add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
    • add links to the "New and changed documentation" section of the latest Migration Guide
  • No: please ensure the base branch is current
  • Unsure: we'll let you know!

Preview

This is used for CodeBlock components, which are used on our /styles page. To test, we want to verify the CodeBlocks are still working.
https://deploy-preview-1063--docs-getdbt-com.netlify.app/styles

@github-actions github-actions bot added the size: x-small This change will take under 3 hours to fix. label Jan 28, 2022
@netlify
Copy link

netlify bot commented Jan 28, 2022

✔️ Deploy Preview for docs-getdbt-com ready!

🔨 Explore the source changes: 1926527

🔍 Inspect the deploy log: https://app.netlify.com/sites/docs-getdbt-com/deploys/61f43ba6429708000776a041

😎 Browse the preview: https://deploy-preview-1063--docs-getdbt-com.netlify.app

@JKarlavige JKarlavige merged commit ae55fd8 into current Jan 31, 2022
@JKarlavige JKarlavige deleted the remove-duplicate-package branch January 31, 2022 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: x-small This change will take under 3 hours to fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants