Skip to content

rename tag-observer exports#253

Merged
keithamus merged 1 commit intomainfrom
rename-tag-observer-exports
May 24, 2022
Merged

rename tag-observer exports#253
keithamus merged 1 commit intomainfrom
rename-tag-observer-exports

Conversation

@keithamus
Copy link
Contributor

When discussing API ergonomics with @koddsson he pointed out that the named exports in tag-observer weren't very clear as to their intention. This renames those APIs to be much clearer.

This isn't a breaking change because none of this is exported, but will be in 2.0.

@keithamus keithamus marked this pull request as ready for review May 24, 2022 13:14
@keithamus keithamus requested a review from a team as a code owner May 24, 2022 13:14
Copy link
Contributor

@koddsson koddsson left a comment

Choose a reason for hiding this comment

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

Looks good!

@keithamus keithamus merged commit d5356b6 into main May 24, 2022
@keithamus keithamus deleted the rename-tag-observer-exports branch May 24, 2022 14:06
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