Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adjust passthrough example #17

Closed
wants to merge 1 commit into from
Closed

Adjust passthrough example #17

wants to merge 1 commit into from

Conversation

ribasushi
Copy link
Contributor

This should work out of the box without further changes with https://metacpan.org/release/RIBASUSHI/DBIx-Class-0.082841_01

@haarg note how I need to do set_mro() in the last hunk, this is something With::Role should preserve on its own.

Note that further adjustments will have to be made for the DBIC version
currently in 'master', but such adjustments are very likely going to be
of comparable complexity to what was needed here. No warranties explicit
or implicit either way.
@ribasushi
Copy link
Contributor Author

Now that 0.082842 shipped, this PR is no longer relevant. Instead https://github.com/dbsrgits/sql-abstract/pull/17/files#diff-eb3465614a3d3abc2ae036e4ba07afcaR30-R45 needs to be packaged and shipped as a standalone dist*

Closing.

*It could be shipped as part of SQLA, but at present this would create a dependency resolution cycle: I am slowly working on ensuring SQLAC drops the SQLA dependncy entirely, but OSS time is extremely scarce

@ribasushi ribasushi closed this Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant