Skip to content

chore(core): removes unused exports#2577

Merged
jorgemoya merged 1 commit intocanaryfrom
remove-unused-exports-core
Sep 5, 2025
Merged

chore(core): removes unused exports#2577
jorgemoya merged 1 commit intocanaryfrom
remove-unused-exports-core

Conversation

@jorgemoya
Copy link
Copy Markdown
Contributor

What/Why?

Removes unused exports from core.

Testing

N/A

Migration

Remove export declarations from functions that are not being imported anywhere else.

@jorgemoya jorgemoya requested a review from a team September 4, 2025 16:34
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Sep 4, 2025

🦋 Changeset detected

Latest commit: 9272a01

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
catalyst-b2b Ready Ready Preview Comment Sep 4, 2025 4:38pm
catalyst-canary Ready Ready Preview Comment Sep 4, 2025 4:38pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
catalyst Ignored Ignored Sep 4, 2025 4:38pm
catalyst-au Ignored Ignored Sep 4, 2025 4:38pm
catalyst-uk Ignored Ignored Sep 4, 2025 4:38pm

});
};

export function assertAddCartLineItemErrors(
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We're not using these asserts, we added these directly to the actions.

}

export function WishlistAnalyticsProviderResolved({
function WishlistAnalyticsProviderResolved({
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I meant these @chanceaclark

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oh that's fine to remove!

@jorgemoya jorgemoya added this pull request to the merge queue Sep 5, 2025
Merged via the queue into canary with commit baf07ca Sep 5, 2025
14 checks passed
@jorgemoya jorgemoya deleted the remove-unused-exports-core branch September 5, 2025 15:02
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