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

Non consistent code (Failed static analysis) prevents development in forked repo #98

Closed
g41797 opened this issue Jan 20, 2024 · 6 comments

Comments

@g41797
Copy link

g41797 commented Jan 20, 2024

What steps will reproduce the problem?

Run "Inspect code" in PHPStorm

What is the expected result?

At least no errors

What do you get instead?

e.g. Error : Call to undefined method Yiisoft\Queue\Message\Message::getId()

Additional info

I tried to fix in forked repo, but getId/setId methods of message interface don't exist in master branch, but exist in another ones etc.

also see status of analysis

@vjik
Copy link
Member

vjik commented Jan 21, 2024

Yes. This package requires to adapt to Yii Queue.

@g41797
Copy link
Author

g41797 commented Jan 21, 2024

let's take getid&setid
half of YiiQueue repositories have these methods
and according to flow id is required

i don't understand who should make this fix

@vjik
Copy link
Member

vjik commented Jan 21, 2024

The work is already underway:

@g41797
Copy link
Author

g41797 commented May 4, 2024

any proof of life

@vjik
Copy link
Member

vjik commented May 6, 2024

any proof of life

The development is in the plans. Unfortunately, there is not enough time right now.

@g41797
Copy link
Author

g41797 commented May 6, 2024

any proof of life

The development is in the plans. Unfortunately, there is not enough time right now.

can i help?

@g41797 g41797 closed this as not planned Won't fix, can't repro, duplicate, stale May 21, 2024
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

No branches or pull requests

2 participants