Skip to content

Added dictionary key support#87

Merged
neuecc merged 2 commits intoCysharp:masterfrom
zorgoz:master
Jul 25, 2025
Merged

Added dictionary key support#87
neuecc merged 2 commits intoCysharp:masterfrom
zorgoz:master

Conversation

@zorgoz
Copy link
Copy Markdown
Contributor

@zorgoz zorgoz commented Nov 21, 2024

See: #86

Added property name handling
@github-actions
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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