Skip to content

remove Chart.js because it's not used. We have the min version

Lauren Barker requested to merge 5323-remove-chart.js into master

Why is this change being made?

This MR removes the following javascripts files from the repo:

/source/javascripts/libs/chart.js

This is done in preparation of using Webpack to bundle javascripts instead of Middleman. These specific javascripts were causing warnings in the webpack build.

Author Checklist

  • Provided a concise title for the MR
  • Added a description to this MR explaining the reasons for the proposed change, per say-why-not-just-what
  • Assign this change to the correct DRI
    • If the DRI for the page/s being updated isn’t immediately clear, then assign it to your manager.
    • If your manager does not have merge rights, please ask someone to merge it AFTER it has been approved by your manager in #mr-buddies.
    • If the changes relate to any part of the project other than updates to content and/or data files please make sure to ping @gl-static-site-editor in a comment for a review and merge. For example changes to .gitlab-ci.yml, JavaScript/CSS/Ruby code or the layout files.
  1. Related #5323 (closed)
  2. Related https://gitlab.com/gitlab-com/marketing/growth-marketing/brand-and-digital/brand-and-digital/-/issues/387
Edited by Lauren Barker

Merge request reports