Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

laravel/framework

crynobone pushed 1 commit to feature-cast-as-json-connection laravel/framework

View on GitHub

crynobone opened a draft pull request on laravel/framework
[11.x] Allow using `castAsJson()` on non default db connection during test
<!-- Please only send a pull request to branches that are currently supported: https://laravel.com/docs/releases#support-policy If you are unsure which branch your pull request should be s...
crynobone created a branch on laravel/framework

feature-cast-as-json-connection - The Laravel Framework.

Sopamo created a review comment on a pull request on laravel/framework
I think it's problematic though that the behaviour is somewhat undefined when passing in a collection (or at least very unexpected), especially as I think many devs don't actively differentiate bet...

View on GitHub

Sopamo created a review on a pull request on laravel/framework

View on GitHub

rodrigopedra created a review comment on a pull request on laravel/framework
You're right. And it is easy enough for someone to just add `collect(...)->all()` on the call-site

View on GitHub

rodrigopedra created a review on a pull request on laravel/framework

View on GitHub

crynobone created a review comment on a pull request on laravel/framework
It might not be even worth it to pursue these changes. Using Collection as value is not in the docs (unless someone can share the URL) and Postgres/SQLServer would require different syntax to store...

View on GitHub

crynobone created a review on a pull request on laravel/framework

View on GitHub

rodrigopedra created a review comment on a pull request on laravel/framework
Also, I'd change the check to `$value instanceof Enumerable`, so `$value->all()` also covers `LazyCollection` instances

View on GitHub

rodrigopedra created a review on a pull request on laravel/framework

View on GitHub

rodrigopedra created a review comment on a pull request on laravel/framework
Why not use `$value->all()` and skip the implicit `array_map` call inside to `Collection@toArray()`?

View on GitHub

rodrigopedra created a review on a pull request on laravel/framework

View on GitHub

github-actions[bot] created a comment on an issue on laravel/framework
Thank you for reporting this issue! As Laravel is an open source project, we rely on the community to help us diagnose and fix issues as it is not possible to research and fix every issue reported...

View on GitHub

crynobone deleted a branch laravel/framework

test-53226

crynobone deleted a branch laravel/framework

fixes-53226

crynobone pushed 1 commit to fixes-53226-1 laravel/framework

View on GitHub

crynobone opened a draft pull request on laravel/framework
[11.x] Fix incorrect bindings in DB::update when using a collection as a value
fixes #53226 <!-- Please only send a pull request to branches that are currently supported: https://laravel.com/docs/releases#support-policy If you are unsure which branch your pull request...
crynobone created a branch on laravel/framework

fixes-53226-1 - The Laravel Framework.

crynobone closed a draft pull request on laravel/framework
[10.x] Test Improvements
Validate #53226
crynobone pushed 1 commit to fixes-53226 laravel/framework

View on GitHub

crynobone pushed 9 commits to fixes-53226 laravel/framework
  • [11.x] Fixes handling `Js::from(collect());` (#53206) Signed-off-by: Mior Muhammad Zaki <[email protected]> f7e2fab
  • fix doc (#53211) 288564d
  • [11.x] fix PHPDoc for \Illuminate\Database\Connection (#53212) * fix phpdoc * fix int -> float 4b0e358
  • Include class-string generics for Validator::$exception (#53210) d2895c7
  • Remove a few useless return void statements. (#53225) a37b40b
  • [11.x] Fixes phpdoc type of Number::forHumans() (#53218) 3ee3ad9
  • [11.x] Fix handling exceptions thrown in eval()'d code (#53204) * [11.x] Fix handling exceptions thrown in eval()'d ... 5a9886c
  • Merge remote-tracking branch 'upstream/10.x' into 11.x Signed-off-by: Mior Muhammad Zaki <[email protected]> 3807ec7
  • Merge branch '11.x' into fixes-53226 d971443

View on GitHub

crynobone pushed 5 commits to 11.x laravel/framework
  • wip 0124751
  • Update version to v10.48.22 c4ea52b
  • Update CHANGELOG 1b3ef8f
  • Ensure headers are only attached to illuminate responses (#53019) 646520a
  • Merge remote-tracking branch 'upstream/10.x' into 11.x Signed-off-by: Mior Muhammad Zaki <[email protected]> 3807ec7

View on GitHub

crynobone pushed 2 commits to fixes-53226 laravel/framework

View on GitHub

taylorotwell pushed 1 commit to fixes-53226 laravel/framework

View on GitHub

crynobone pushed 1 commit to fixes-53226 laravel/framework

View on GitHub

crynobone pushed 8 commits to fixes-53226 laravel/framework

View on GitHub

crynobone pushed 1 commit to test-53226 laravel/framework

View on GitHub

crynobone pushed 1 commit to test-53226 laravel/framework

View on GitHub

crynobone created a comment on an issue on laravel/framework
https://github.com/laravel/framework/pull/53253 Unable to verify this issue. Tests running fine here.

View on GitHub

Load more