search

pksunkara / pgx_ulid

Postgres extension for ulid
MIT License
346 stars 21 forks source link

Update dependencies #40

Closed arjunyel closed 6 months ago

pksunkara commented 6 months ago

Why updating the MSRV?

And you are changing the default feature which is a breaking change and not a minor version upgrade like you have.

arjunyel commented 6 months ago

@pksunkara hi friend, sorry I am new to Rust, is there anything wrong with requiring the latest rust since it will just be built from the GitHub? Is there anything I can change to get this merged? Thanks!

pksunkara commented 6 months ago

is there anything wrong with requiring the latest rust since it will just be built from the GitHub

No, but it's not needed either.

Updating pgrx is also not needed since it's better to wait for 0.12.0 release