Skip to content

Conversation

@chr-hertel
Copy link
Contributor

Was running into a few deprecations - pretty sure that's not all

@CodeWithKyrian
Copy link
Owner

You're right. That's not all. I'll add more commits to this PR before merging. Thanks for the contribution

@CodeWithKyrian
Copy link
Owner

Well, it looks like I don’t have permission to push to the branch, which is fine.

Could you update line 15 of Codewithkyrian\ChromaDB\Generated\Exceptions\ValidationException to \Throwable|null $previous = null)? That should cover everything. Thanks!

@chr-hertel chr-hertel force-pushed the ci-php84-deprectations branch from 1d70c23 to 9145103 Compare March 9, 2025 14:34
@CodeWithKyrian CodeWithKyrian merged commit 61f4baa into CodeWithKyrian:main Mar 9, 2025
8 checks passed
@chr-hertel chr-hertel deleted the ci-php84-deprectations branch March 9, 2025 17:47
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