CodSpeed Performance Report

Merging #4831 will not alter performance

_{Comparing pranayat:feat/shadow-same-as-direct (22344d2) with main (612a7f7)}

Summary

✅ 11 untouched benchmarks

Hey @pranayat! thanks for the PR :)
I took a look and I'm only getting this validation error to appear when I have all three url properties defined.

Error appears on both url and directUrl here when they're both defined

But does not appear at all here when only url and shadowDatabaseUrl are defined - A datasource without url leads to other validation errors :)

@Druue Ah good catch! I've spotted the issue in my code, shall fix it :)

I've pushed a commit which I hopes passes the CI checks. Ever since I synced my branch with upstream two weeks ago, local builds make my system go out of disk space, so I'm no longer able to run tests locally.

@Druue Can review now :)

✨ 🎉

Late to the party (so feel free to ignore!), but how about adding ... to avoid data loss. to the end of both error messages to give users some explanation why as well?

I'll most certainly not ignore this 😄, could save users a google search.

Awesome! Thank you so much! :D ❤️

Thank you for the great review 😄