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

chore (providers): Remove obsolete 'internal' from several packages. #3901

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

shaper
Copy link
Contributor

@shaper shaper commented Nov 26, 2024

…ackages.

@shaper shaper requested a review from lgrammel November 26, 2024 20:37
@shaper shaper changed the title chore (providers): Remove obsolete 'internal' from several provider p… chore (providers): Remove obsolete 'internal' from several packages. Nov 26, 2024
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Seems ambiguous re: whether we would want a changeset for this. It's effectively internal cleanup, but it's nice to publish it so that the public code reflects the cleanliness.

@lgrammel lgrammel merged commit b1f31da into main Nov 27, 2024
9 checks passed
@lgrammel lgrammel deleted the shaper/pr/rm-internal branch November 27, 2024 08:26
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