[DBIx-Class-Devel] [dbsrgits/sql-translator] Oracle producer add missing functions (PR #143)

hazardv notifications at github.com
Thu Feb 16 21:18:06 GMT 2023


> I'm a bit surprised by the tests being removed from the XML-related tests. If @rabbiveesh is content it's for a good reason then so am I. Otherwise, all looks fine.

@mohawk2  The only things I removed from the XML tests were the checks looking for the triggers that previously got created for all timestamp fields. Since there is no reason to create that trigger for all timestamp fields I commented out the code in commit https://github.com/dbsrgits/sql-translator/pull/143/commits/ab42ef849111e4091ccae0981bb4241d60c15146. I then removed the commented-out code in commit https://github.com/dbsrgits/sql-translator/pull/143/commits/b14da74a5bb7e2e342e2651c6abfdf656e153128. 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/dbsrgits/sql-translator/pull/143#issuecomment-1433726585
You are receiving this because you are subscribed to this thread.

Message ID: <dbsrgits/sql-translator/pull/143/c1433726585 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scsys.co.uk/pipermail/dbix-class-devel/attachments/20230216/46495011/attachment.htm>


More information about the DBIx-Class-Devel mailing list