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

Reduce number of dependencies #8790

Merged
merged 17 commits into from
Aug 30, 2023
Merged

Reduce number of dependencies #8790

merged 17 commits into from
Aug 30, 2023

Conversation

dcousens
Copy link
Member

As the title suggests, this pull request reduces the number of dependencies used by packages/core/ and our tests/.
A number of listed dependencies were unused, only used for development, or otherwise probably unnecessary.

@dcousens dcousens requested a review from borisno2 August 30, 2023 01:22
@dcousens dcousens self-assigned this Aug 30, 2023
@dcousens dcousens force-pushed the less-deps branch 3 times, most recently from d772785 to 1356a10 Compare August 30, 2023 01:57
@dcousens dcousens force-pushed the less-deps branch 2 times, most recently from 5238611 to 2ef5740 Compare August 30, 2023 02:00
@dcousens dcousens changed the title Reduce number of dependencies Reduce number of dependencies, remove resolved workaround Aug 30, 2023
@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 30, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 3e22925:

Sandbox Source
@keystone-6/sandbox Configuration

@dcousens dcousens changed the title Reduce number of dependencies, remove resolved workaround Reduce number of dependencies Aug 30, 2023
@dcousens dcousens merged commit b830b7c into main Aug 30, 2023
@dcousens dcousens deleted the less-deps branch August 30, 2023 11:55
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