Merge pull request #288 from dcastil/breaking-feature/155/remove-depr…

…ecated-exports Remove deprecated exports
dcastil · Aug 19, 2023 · cb8fabe · cb8fabe
2 parents f31447b + a4134f1
commit cb8fabe
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 24 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -1,5 +1,3 @@
-import { twJoin } from './lib/tw-join'
-
 export { createTailwindMerge } from './lib/create-tailwind-merge'
 export { getDefaultConfig } from './lib/default-config'
 export { extendTailwindMerge } from './lib/extend-tailwind-merge'
@@ -9,8 +7,3 @@ export { twJoin, type ClassNameValue } from './lib/tw-join'
 export { twMerge } from './lib/tw-merge'
 export type { Config } from './lib/types'
 export * as validators from './lib/validators'
-
-/**
- * @deprecated Will be removed in next major version. Use `twJoin` instead.
- */
-export const join = twJoin
diff --git a/src/lib/validators.ts b/src/lib/validators.ts
@@ -36,11 +36,6 @@ export function isArbitraryNumber(value: string) {
     return getIsArbitraryValue(value, 'number', isNumber)
 }
 
-/**
- * @deprecated Will be removed in next major version. Use `isArbitraryNumber` instead.
- */
-export const isArbitraryWeight = isArbitraryNumber
-
 export function isNumber(value: string) {
     return !Number.isNaN(Number(value))
 }

diff --git a/tests/public-api.test.ts b/tests/public-api.test.ts
@@ -5,7 +5,6 @@ import {
     extendTailwindMerge,
     fromTheme,
     getDefaultConfig,
-    join,
     mergeConfigs,
     twJoin,
     twMerge,
@@ -25,7 +24,6 @@ test('has correct export types', () => {
         isArbitrarySize: expect.any(Function),
         isArbitraryUrl: expect.any(Function),
         isArbitraryValue: expect.any(Function),
-        isArbitraryWeight: expect.any(Function),
         isInteger: expect.any(Function),
         isLength: expect.any(Function),
         isPercent: expect.any(Function),
@@ -34,7 +32,7 @@ test('has correct export types', () => {
     })
     expect(mergeConfigs).toStrictEqual(expect.any(Function))
     expect(extendTailwindMerge).toStrictEqual(expect.any(Function))
-    expect(join).toStrictEqual(expect.any(Function))
+    expect(twJoin).toStrictEqual(expect.any(Function))
 
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     const noRun = () => {
@@ -149,7 +147,6 @@ test('validators have correct inputs and outputs', () => {
     expect(validators.isArbitrarySize('')).toEqual(expect.any(Boolean))
     expect(validators.isArbitraryPosition('')).toEqual(expect.any(Boolean))
     expect(validators.isArbitraryUrl('')).toEqual(expect.any(Boolean))
-    expect(validators.isArbitraryWeight('')).toEqual(expect.any(Boolean))
     expect(validators.isArbitraryNumber('')).toEqual(expect.any(Boolean))
     expect(validators.isArbitraryShadow('')).toEqual(expect.any(Boolean))
 })
@@ -181,14 +178,6 @@ test('fromTheme has correct inputs and outputs', () => {
     expect(fromTheme('foo')({ foo: ['hello'] })).toStrictEqual(['hello'])
 })
 
-test('join has correct inputs and outputs', () => {
-    expect(join()).toStrictEqual(expect.any(String))
-    expect(join('')).toStrictEqual(expect.any(String))
-    expect(join('', [false, null, undefined, 0, [], [false, [''], '']])).toStrictEqual(
-        expect.any(String),
-    )
-})
-
 test('twJoin has correct inputs and outputs', () => {
     expect(twJoin()).toStrictEqual(expect.any(String))
     expect(twJoin('')).toStrictEqual(expect.any(String))