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

Update AnnotationScanner.php #22

Closed
weierophinney opened this issue Dec 31, 2019 · 1 comment · Fixed by #58
Closed

Update AnnotationScanner.php #22

weierophinney opened this issue Dec 31, 2019 · 1 comment · Fixed by #58
Assignees
Labels
Bug Something isn't working Won't Fix This will not be worked on
Milestone

Comments

@weierophinney
Copy link
Member

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Original Issue: https://api.github.com/repos/zendframework/zendframework/issues/7113
User: @nouc23
Created On: 2015-01-09T10:38:17Z
Updated At: 2015-05-07T14:11:59Z
Body
I know we should write code using PSR standard but life is different.
While we use tabs script ignore annotations.

Fixes #5595


Comment

User: @Ocramius
Created On: 2015-01-09T10:38:57Z
Updated At: 2015-01-09T10:38:57Z
Body
This requires a test asset containing tabs, and a related test


Comment

User: @nouc23
Created On: 2015-01-09T10:58:53Z
Updated At: 2015-01-09T10:58:53Z
Body
done


Comment

User: @weierophinney
Created On: 2015-02-19T20:50:49Z
Updated At: 2015-02-19T20:50:49Z
Body
Needs:

  • Rebase
  • new test.

Comment

User: @Maks3w
Created On: 2015-05-07T13:57:11Z
Updated At: 2015-05-07T13:57:11Z
Body
I've pulled this and found the test is not related with the change made in scanner. So I don't know what is trying to fix this. Test never fails.


Comment

User: @nouc23
Created On: 2015-05-07T14:11:59Z
Updated At: 2015-05-07T14:11:59Z
Body

I edit existing test, and I was told to create new one.

I forgot to do that and also I doesnt use github much. I try to finish this up on days.

Wysłano z aplikacji myMail dla Androida
czwartek, 07 maja 2015, 03:57PM +02:00 od Maks3w [email protected]:

I've pulled this and found the test is not related with the change made in scanner. So I don't know what is trying to fix this. Test never fails.

Reply to this email directly or  view it on GitHub .



Originally posted by @GeeH at zendframework/zend-code#78

@Ocramius Ocramius added Bug Something isn't working Won't Fix This will not be worked on labels Nov 30, 2020
@Ocramius Ocramius self-assigned this Nov 30, 2020
@Ocramius Ocramius added this to the 4.0.0 milestone Nov 30, 2020
@Ocramius
Copy link
Member

No longer needed as per #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Won't Fix This will not be worked on
Projects
None yet
2 participants