Skip to content
This repository was archived by the owner on Jul 14, 2026. It is now read-only.

Optimizations - #16

Open
rufhausen wants to merge 29 commits into
BWICompanies:masterfrom
rufhausen:optimizations
Open

Optimizations#16
rufhausen wants to merge 29 commits into
BWICompanies:masterfrom
rufhausen:optimizations

Conversation

@rufhausen

Copy link
Copy Markdown

No description provided.

@bennettblack

Copy link
Copy Markdown
Contributor

Thanks for the PR,

  1. Why have you changed the package namespace to your own?
  2. This looks to break backwards compatibility with previous versions of Laravel.

@rufhausen

Copy link
Copy Markdown
Author

@bennettblack Hey, sorry, I had no intention of making a PR back to your repository. I apologize for maybe not understanding how the forking would work. I was just trying to get something working for Laravel 12 as a POC, and I allowed AI go run wild in the background at some point (for all I know the tests added are all gibberish). If what I did was against any licensing, etc., I apologize. You can close this PR.

@bennettblack

Copy link
Copy Markdown
Contributor

@rufhausen No worries! I understand. I'm very interested in merging in some of this functionality once you get something working for V12.

@rufhausen

Copy link
Copy Markdown
Author

@bennettblack Everything seems to be working well based on playing around in Tinker with some Eloquent models and relationships, etc.
I'm connecting to a remote DB2 instanced via VPN and it seems pretty performant. As I mentioned, I relied on AI for most of this, partly out of curiosity to see what it would do and partly out not having any real experience directly with DB2. It purported to find and add some missing functionality, including for Upserts as an example.
I would certainly prefer to use an updated version your package. I just needed to try getting some proof of concept working and we are on Laravel 12.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants