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

Rename join to twJoin #161

Merged
merged 6 commits into from
Oct 20, 2022
Merged

Rename join to twJoin #161

merged 6 commits into from
Oct 20, 2022

Conversation

dcastil
Copy link
Owner

@dcastil dcastil commented Oct 20, 2022

The reason being that twJoin is easier to refactor with search and replace than join which is a function name used commonly, e.g. in lodash or Array.prototype.

join function stays in library exports but gets deprecated. I'll remove it in the next major version. Should add that info to changelog.

@dcastil dcastil added the needs changelog edit Needs edit in changelog before release label Oct 20, 2022
@github-actions github-actions bot added the feature Is new feature label Oct 20, 2022
@github-actions
Copy link

size-limit report 📦

Path Size
dist/tailwind-merge.mjs 6.29 KB (+0.05% 🔺)
dist/tailwind-merge.cjs.production.min.js 6.29 KB (+0.1% 🔺)

@dcastil dcastil merged commit e9f1e1b into main Oct 20, 2022
@dcastil dcastil deleted the feature/rename-join-to-twJoin branch October 20, 2022 07:19
@github-actions
Copy link

github-actions bot commented Nov 5, 2022

This was addressed in release v1.8.0.

@dcastil dcastil added the context-v1 Related to tailwind-merge v1 label Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
context-v1 Related to tailwind-merge v1 feature Is new feature needs changelog edit Needs edit in changelog before release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant