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

Fix arbitrary length 0 without unit not being recognized #237

Merged
merged 2 commits into from
May 31, 2023

Conversation

dcastil
Copy link
Owner

@dcastil dcastil commented May 31, 2023

Closes #234

@github-actions github-actions bot added the bugfix Is bug fix label May 31, 2023
@dcastil dcastil changed the title Fix arbitrary length 0 not being recognized Fix arbitrary length 0 without unit not being recognized May 31, 2023
@github-actions
Copy link

size-limit report 📦

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

@dcastil dcastil merged commit 21e2b9d into main May 31, 2023
@dcastil dcastil deleted the bugfix/234/fix-arbitrary-length-0-not-recognized branch May 31, 2023 21:16
@github-actions
Copy link

github-actions bot commented Jun 1, 2023

This was addressed in release v1.13.0.

gjtorikian added a commit to gjtorikian/tailwind_merge that referenced this pull request Jun 2, 2023
sebald referenced this pull request in sebald/pattern-analyzer Jun 19, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [tailwind-merge](https://togithub.com/dcastil/tailwind-merge) |
[`1.12.0` ->
`1.13.1`](https://renovatebot.com/diffs/npm/tailwind-merge/1.12.0/1.13.1)
|
[![age](https://badges.renovateapi.com/packages/npm/tailwind-merge/1.13.1/age-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://badges.renovateapi.com/packages/npm/tailwind-merge/1.13.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://badges.renovateapi.com/packages/npm/tailwind-merge/1.13.1/compatibility-slim/1.12.0)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://badges.renovateapi.com/packages/npm/tailwind-merge/1.13.1/confidence-slim/1.12.0)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>dcastil/tailwind-merge</summary>

###
[`v1.13.1`](https://togithub.com/dcastil/tailwind-merge/releases/tag/v1.13.1)

[Compare
Source](https://togithub.com/dcastil/tailwind-merge/compare/v1.13.0...v1.13.1)

##### Bug Fixes

- Fix `basis-auto` and `basis-full` not being merged correctly by
[@&#8203;lukasz-kapica](https://togithub.com/lukasz-kapica) in
[https:/dcastil/tailwind-merge/pull/242](https://togithub.com/dcastil/tailwind-merge/pull/242)

**Full Changelog**:
dcastil/tailwind-merge@v1.13.0...v1.13.1

###
[`v1.13.0`](https://togithub.com/dcastil/tailwind-merge/releases/tag/v1.13.0)

[Compare
Source](https://togithub.com/dcastil/tailwind-merge/compare/v1.12.0...v1.13.0)

This release focuses on improvements to the docs.

##### New Features

- Add intro video from Simon Vrachliotis to docs by
[@&#8203;dcastil](https://togithub.com/dcastil) in
[https:/dcastil/tailwind-merge/pull/239](https://togithub.com/dcastil/tailwind-merge/pull/239)
- [@&#8203;simonswiss](https://togithub.com/simonswiss) made an intro
video to tailwind-merge which I added to the docs. Check it out
[here](https://togithub.com/dcastil/tailwind-merge/blob/v1.13.0/docs/what-is-it-for.md#video-introduction)!
- Add docs about when to use tailwind-merge by
[@&#8203;dcastil](https://togithub.com/dcastil) in
[https:/dcastil/tailwind-merge/pull/230](https://togithub.com/dcastil/tailwind-merge/pull/230)
- I added a new page to the docs about [when and how to use
tailwind-merge](https://togithub.com/dcastil/tailwind-merge/blob/v1.13.0/docs/when-and-how-to-use-it.md)
which should help you to decide whether you should use it and what
alternative approaches exist.
- Make it clearer in docs that tailwind-merge supports composition of
class strings by [@&#8203;dcastil](https://togithub.com/dcastil) in
[https:/dcastil/tailwind-merge/pull/229](https://togithub.com/dcastil/tailwind-merge/pull/229)
- Add npm package provenance by
[@&#8203;dcastil](https://togithub.com/dcastil) in
[https:/dcastil/tailwind-merge/pull/219](https://togithub.com/dcastil/tailwind-merge/pull/219)
- GitHub introduced a new security feature to verify which source commit
and build file were used for a specific npm package version.
tailwind-merge now publishes provenance signatures alongside all
releases on npm. [Read more on the GitHub
blog.](https://github.blog/2023-04-19-introducing-npm-package-provenance/)

##### Bug Fixes

- Fix arbitrary length `0` without unit not being recognized by
[@&#8203;dcastil](https://togithub.com/dcastil) in
[https:/dcastil/tailwind-merge/pull/237](https://togithub.com/dcastil/tailwind-merge/pull/237)
- Fix typo in comment in default-config.ts by
[@&#8203;CrutchTheClutch](https://togithub.com/CrutchTheClutch) in
[https:/dcastil/tailwind-merge/pull/227](https://togithub.com/dcastil/tailwind-merge/pull/227)

**Full Changelog**:
dcastil/tailwind-merge@v1.12.0...v1.13.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://app.renovatebot.com/dashboard#github/sebald/pattern-analyzer).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMTAuMCIsInVwZGF0ZWRJblZlciI6IjM1LjExMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@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
bugfix Is bug fix context-v1 Related to tailwind-merge v1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tailwind merge didn't merge custom value with only 0 unit
1 participant