Skip to content

Conversation

@jonaspm
Copy link
Contributor

@jonaspm jonaspm commented May 17, 2025

Upgraded drizzle-orm to latest v0.42.0

@changeset-bot
Copy link

changeset-bot bot commented May 17, 2025

🦋 Changeset detected

Latest commit: d7339b2

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jonaspm
Copy link
Contributor Author

jonaspm commented May 23, 2025

@ematipico hello Emanuele! do you have any feedback or can we merge this?

@ematipico
Copy link
Member

I do. Other than upgrading the library, there are changes that you haven't explained. Can you please update the PR, and describe those changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this change was made due to a change in Drizzle, indexes are now expected to be an array instead of an object.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added new properties from Drizzle column

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fix for incompatible types

@ematipico ematipico merged commit 03435f8 into withastro:main May 26, 2025
15 checks passed
@astrobot-houston astrobot-houston mentioned this pull request May 26, 2025
openscript pushed a commit to openscript/astro that referenced this pull request Sep 12, 2025
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.

2 participants