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

Package json cleanup #600

Merged
merged 7 commits into from
Nov 13, 2023
Merged

Package json cleanup #600

merged 7 commits into from
Nov 13, 2023

Conversation

sdadn
Copy link
Contributor

@sdadn sdadn commented Nov 9, 2023

Summary

Additional Details

This PR cleans up package.json by:

  • Enabling automatic peer Dependency installation.
  • Removing all dependencies that aren't used directly within this repo. All those dependencies are installed automatically as the peer dependencies of terra components such as react, react-intl etc.
  • Removing all terra components. Adding them was redundant as they were automatically installed as sub dependencies of the docs packages.
  • Using the prettifyJSON script script to format the package.json file.
  • Deleting the unused Heroku scripts.

This PR also drops npm 6 support as it now relies on automatic peer dependency installation.

Testing

This was tested by doing a clean install after removing dependencies and running npm start to ensure that the dev site builds and has the same content.

Thank you for contributing to Terra.
@cerner/terra

@sdadn sdadn self-assigned this Nov 9, 2023
@github-actions github-actions bot temporarily deployed to preview-pr-600 November 9, 2023 18:37 Destroyed
@sdadn sdadn requested a review from a team November 9, 2023 18:38
Copy link

@adoroshk adoroshk left a comment

Choose a reason for hiding this comment

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

Tested locally by running npm run clean:install and npm run start commands. Workes as expected.
Screenshot 2023-11-09 at 2 06 58 PM

@sdadn sdadn merged commit ba56fdd into main Nov 13, 2023
3 checks passed
@sdadn sdadn deleted the package-json-cleanup branch November 13, 2023 14:31
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