Skip to content
This repository was archived by the owner on Jan 21, 2026. It is now read-only.

f/matej-utils - add: numberComparator#10

Open
matej-smid-cleevio wants to merge 1 commit into
mainfrom
f/matej-utils
Open

f/matej-utils - add: numberComparator#10
matej-smid-cleevio wants to merge 1 commit into
mainfrom
f/matej-utils

Conversation

@matej-smid-cleevio

Copy link
Copy Markdown

Adds a new battle-tested function numberComparator that simplifies the sorting of items when the sorting criteria are more complex than just sorting alphabetically or numerically.

I'm open to criticism, renaming and improvement.

@matej-smid-cleevio matej-smid-cleevio self-assigned this Jun 12, 2025
@matej-smid-cleevio matej-smid-cleevio added the enhancement New feature or request label Jun 12, 2025
@lukasriha

Copy link
Copy Markdown
Member

Budes muset jeste vygenerovt ten changeset - to udelas pomoci pnpm changeset add --since. Na scope a package uverim ze prijdes sam :)

@lukasriha lukasriha left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Formatovat pomoci pnpm format:fix a pridat changelog aby se mohla releasenout verze :)

Taky pridej testy prosim - kdyz budeme odsud distrubovt utilitni funkce tak musi byt naprosto neprustrelne

@matej-smid-cleevio

Copy link
Copy Markdown
Author
  • naformátuju to,
  • přidal bys sem nějaký example test? jak ty testy spustím?
  • a ještě jsem si všiml že by bylo asi good tu funkci exportovat z index.ts, tak to přidám

@lukasriha

Copy link
Copy Markdown
Member
  • a ještě jsem si všiml že by bylo asi good tu funkci exportovat z index.ts, tak to přidám

Presne, musi to byt vyexportovne z index.ts abys to mohl pouzivat

import { numberComparator } from '@cleevio/ts-utils';

Alternativne muzes exportovat ten file pomoci exports fieldu v package.json ale to nechceme

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants