Skip to content

Conversation

@mspirkov
Copy link
Contributor

@mspirkov mspirkov commented Nov 2, 2025

Q A
Is bugfix? ✔️
New feature?
Breaks BC?
Fixed issues #313

When running the ./yii api/generate 2.0 command in yiiframework.com I got an error:

image

After this fix, everything worked.

image

@terabytesoftw
Copy link
Member

Update actions so that the tests can be run.

replace:

  • actions/cache@v1 => actions/cache@v4
  • actions/checkout@v2 => actions/checkout@v5

Error: This request has been automatically failed because it uses a deprecated version of actions/cache: v1. Please update your workflow to use v3/v4 of actions/cache to avoid interruptions. Learn more: https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down

@mspirkov mspirkov marked this pull request as ready for review November 2, 2025 09:34
@samdark
Copy link
Member

samdark commented Nov 2, 2025

@mspirkov is this part covered by a test? We still run PHP 7.4.33 at yiiframework.com. Would it be OK with that version?

@mspirkov
Copy link
Contributor Author

mspirkov commented Nov 2, 2025

I got this error using PHP 7.4.33 in yiiframework.com

We still run PHP 7.4.33 at yiiframework.com. Would it be OK with that version?

@mspirkov
Copy link
Contributor Author

mspirkov commented Nov 2, 2025

After this fix, I checked the generated documentation and it seems to be normal.

@mspirkov
Copy link
Contributor Author

mspirkov commented Nov 2, 2025

As far as I can see, the tests don't cover this scenario. I'll try to fix it now.

@mspirkov
Copy link
Contributor Author

mspirkov commented Nov 2, 2025

@samdark tests added

@samdark samdark merged commit 59ad069 into yiisoft:master Nov 3, 2025
4 checks passed
@samdark
Copy link
Member

samdark commented Nov 3, 2025

Thank you!

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.

3 participants