-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Test SQL Server 17 on Travis #3050
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
Conversation
5275fed
to
e82cc4c
Compare
@Majkl578 Microsoft accepts their own EULA same way on Travis, so I think it should be OK. Maybe we could read the agreement and check what we're agreeing to :) I'm seeing exactly the same failures on Windows and Linux (#2617 (comment)) so hopefully I can fix them soon. |
Btw sqlsrv extension fails to compile with nightly, but we can ignore it imho. |
I’d say we can disable testing all PECL-based drivers with nightly PHP since it’s a normal thing. |
Ok, nightly stages removed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢
Finally managed to solve the problems with running SQL Server 17 on Travis.
The tests are green for the
pdo_sqlsrv
driver.Tnere are two failures for the
sqlsrv
driver:@morozov Do you think you can help with these?
I'm not quite sure if this is legally ok to run it on CI & auto-accept EULA though, any opinions?
Not up to date with master due to #3049.