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

Remove extraneous packages from lock file #2121

Closed
wants to merge 1 commit into from

Conversation

m-akinc
Copy link
Contributor

@m-akinc m-akinc commented May 17, 2024

Pull Request

🀨 Rationale

Found old, extraneous entries in our lock file from before the Angular workspace got moved.

πŸ‘©β€πŸ’» Implementation

Removed them.

πŸ§ͺ Testing

N/A

βœ… Checklist

  • I have updated the project documentation to reflect my changes or determined no changes are needed.

@rajsite
Copy link
Member

rajsite commented May 17, 2024

Can you comment on how they were removed? We shouldn't be hand editing the lock file

@rajsite
Copy link
Member

rajsite commented May 20, 2024

Closing in favor of #2123

@rajsite rajsite closed this May 20, 2024
rajsite added a commit that referenced this pull request May 20, 2024
# Pull Request

## 🀨 Rationale

It was identified in #2121 that the angular refactor resulted in
leftover entries in the lock file. The entries were marked `extraneous`
and did not seem to interfere with the build, etc. This PR rebuilds the
lock file which cleans out extraneous entries and updates dependencies
to package.json supported range latest.

## πŸ‘©β€πŸ’» Implementation

```
git clean -fdx
del package-lock.json
npm install
```

## πŸ§ͺ Testing

Rely on CI.

## βœ… Checklist

- [x] I have updated the project documentation to reflect my changes or
determined no changes are needed.
@rajsite rajsite deleted the users/makinc/fix-lock-file branch May 20, 2024 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants