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

Allow php 8.3 #410

Merged
merged 7 commits into from
Sep 18, 2023
Merged

Allow php 8.3 #410

merged 7 commits into from
Sep 18, 2023

Conversation

fezfez
Copy link
Contributor

@fezfez fezfez commented Sep 18, 2023

No description provided.

@fezfez
Copy link
Contributor Author

fezfez commented Sep 18, 2023

mutation fail but not due to this PR

@Ocramius
Copy link
Collaborator

It looks like most of them are around how exceptions look like: more testing addions, perhaps?

We could also remove the 0 by using the previous: named parameter: ugly, but usable.

@Ocramius Ocramius added this to the 4.7.0 milestone Sep 18, 2023
@Ocramius
Copy link
Collaborator

Please wait for #411 before continuing, as @fredden is already on it :D

@Ocramius
Copy link
Collaborator

@fezfez rebasing should solve CI issues here

@fezfez
Copy link
Contributor Author

fezfez commented Sep 18, 2023

@fezfez rebasing should solve CI issues here

done

@Ocramius
Copy link
Collaborator

Oof, still one mutant left 😰

@fredden
Copy link
Contributor

fredden commented Sep 18, 2023

Oof, still one mutant left 😰

This is the one squashed in #412. I think we should merge in #388 and then get this rebased as several of the changes here are already being done there.

@Ocramius
Copy link
Collaborator

Done BTW: more rebasing work 😬

@fezfez
Copy link
Contributor Author

fezfez commented Sep 18, 2023

Done BTW: more rebasing work 😬

done

phpunit.xml.dist Outdated
processIsolation="false"
backupGlobals="false"
>
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd" bootstrap="./vendor/autoload.php" colors="true" stopOnFailure="false" processIsolation="false" backupGlobals="false" cacheDirectory=".phpunit.cache">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we keep the old formatting here? I find it easier to read with the previous layout.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks; that helps. :)

phpunit.xml.dist Outdated
backupGlobals="false"
>
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why has this namespace schema location changed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is produced by using the command provided by phpunit 10 (--migrate-schema)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should reference the vendor path here, not a WWW URL

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Collaborator

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, one more 😬

phpunit.xml.dist Outdated
backupGlobals="false"
>
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should reference the vendor path here, not a WWW URL

@Ocramius
Copy link
Collaborator

Let's also check if CI is green 😁

@fezfez
Copy link
Contributor Author

fezfez commented Sep 18, 2023

all seem good now :)

Copy link
Collaborator

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @fezfez!

@Ocramius Ocramius self-assigned this Sep 18, 2023
@Ocramius Ocramius merged commit a80a012 into maglnet:4.7.x Sep 18, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants