Skip to content

Conversation

zorgoz
Copy link
Contributor

@zorgoz zorgoz commented Nov 21, 2024

See: #86

Added property name handling
Copy link
Contributor

This PR is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label May 21, 2025
@zorgoz
Copy link
Contributor Author

zorgoz commented May 21, 2025

Is this repo abandoned?

@github-actions github-actions bot removed the stale label May 22, 2025
@Vectorial1024
Copy link

I have no relationships with CyGames (the owner of this repo), but by cross-referencing with other online sources, I can see the UUID specs now have a spec for UUID v7, which is basically the ULID spec except a few bits are taken away to denote the UUID version (as per usual UUID spec) and the variant.

With this, as more and more systems start to support UUIDv7 (e.g. database engines offering UUID-optimized indexing, existing UUID libraries adopting the UUIDv7, etc), perhaps this "ULID" concept may actually become legacy code.

ULID was created because the UUIDs available back then simply did not have time-based natural sorting, which causes a lot of pain (e.g. extra painful indexing when mass-inserting to the primary key field). My opinion is that, for new systems, perhaps it is better to just use UUIDv7 instead of ULID.

I can see why this package is suddenly "abandoned".

@neuecc
Copy link
Member

neuecc commented Jul 25, 2025

Regarding Ulid vs UUIDv7, I've mentioned this before, but I'm more positive about adopting UUIDv7.
#85 (comment)
Basically, I think the Ulid specification has served its purpose.

However, thank you for the PR, and sorry for leaving it without a response.
I think this PR itself is useful, so considering the slight behavior change, I'll release it as a minor version update (1.4.0).

@neuecc neuecc merged commit 4ca4482 into Cysharp:master Jul 25, 2025
@zorgoz
Copy link
Contributor Author

zorgoz commented Jul 25, 2025 via email

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.

3 participants