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

Updated composer depedencies to work with Codeception 5 #523

Merged
merged 25 commits into from
Jun 17, 2022
Merged

Updated composer depedencies to work with Codeception 5 #523

merged 25 commits into from
Jun 17, 2022

Conversation

developedsoftware
Copy link
Contributor

Q A
Is bugfix? o
New feature? no
Breaks BC? no
Tests pass? unknown

@developedsoftware
Copy link
Contributor Author

Not sure if this can even be done yet - as codception 5 requires PHP 8.

Yii2 I assume will support PHP 7 for quite some time...should I just close this?

@samdark
Copy link
Member

samdark commented Mar 30, 2022

@developedsoftware I think it's possible to allow both. Is there a stable release already?

@developedsoftware
Copy link
Contributor Author

Yes I believe so. I will look to do that ! Thanks

I always think composer is more linear than it is !

@developedsoftware
Copy link
Contributor Author

OK this should be good to go now!

@developedsoftware
Copy link
Contributor Author

Requesting approval if possible.

@bizley
Copy link
Member

bizley commented Jun 15, 2022

I'm worried about all the dev dependencies here...

@developedsoftware
Copy link
Contributor Author

I'm worried about all the dev dependencies here...

No different to 2.0.45?

Just version numbers have been altered to allow for the codeception 5 depedencies.

What would you like me to alter?

@@ -4,6 +4,6 @@ include:
- frontend
- backend
paths:
log: console/runtime/logs
output: console/runtime/output
Copy link

Choose a reason for hiding this comment

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

is this needed, log seems just fine - and could potentially break some existing stuff?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

log is dropped in favour of output in codeception 2.3 so there should not be any breaking changes (all my tests passed locally)

See here Codeception/Codeception#4688 (comment)

My understanding was that log was transposed to output in the background for some time (for backward compatibilty before being dropped altogether which causes it to now error)

Original issue -> Codeception/Codeception#4688

@bizley
Copy link
Member

bizley commented Jun 15, 2022

I mean ^5.0.0-alpha2 or 9.5.x-dev or dev-codecept5

@developedsoftware
Copy link
Contributor Author

developedsoftware commented Jun 15, 2022

I mean ^5.0.0-alpha2 or 9.5.x-dev or dev-codecept5

I dont follow?

9.5.x-dev is for phpunit/phpunit?

dev-codecept5 is for codeception/module-yii2 which requires "codeception/codeception": "^5.0.0-alpha2"

which is also the requirement for codeception/codeception in this repo ^5.0.0-alpha2

So to me that all checks out OK? Can you clarify?

@developedsoftware
Copy link
Contributor Author

developedsoftware commented Jun 15, 2022

I'm worried about all the dev dependencies here...

I know what you mean when you said you worry about them dev dependencies. Yes I agree it would be nice to have a stable releases but there arent any so I am forced to use the dev branches.

See here -> Codeception/Codeception#6327

Not sure how I can request a new release of the yii2 module - so the dev branch is the only way -> https://github.com/Codeception/module-yii2/releases

@samdark samdark merged commit 62c79b3 into yiisoft:master Jun 17, 2022
@samdark
Copy link
Member

samdark commented Jun 17, 2022

Thanks.

@samdark samdark added this to the 2.0.46 milestone Jun 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants