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

chore: drop depedency php-parser #36393

Merged
merged 1 commit into from
Feb 2, 2023
Merged

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Jan 26, 2023

  • Resolves: /

Summary

Dear php-parser,

You were a great help to check that apps are compliant and we did not forget the since and deprecated annotations.
Thank you for all the service you provided us over the years.

TODO

Checklist

@kesselb kesselb self-assigned this Jan 26, 2023
@kesselb
Copy link
Contributor Author

kesselb commented Jan 27, 2023

Check CI

Failing integration-sharing-v1-video-verification should be fixed by #36389.

Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

Currently brings in Symfony Mailer change. So this is blocked by #30349

@kesselb
Copy link
Contributor Author

kesselb commented Jan 27, 2023

Thanks, added a Todo to wait for #30349.

@blizzz blizzz mentioned this pull request Feb 2, 2023
@blizzz
Copy link
Member

blizzz commented Feb 2, 2023

Thanks, added a Todo to wait for #30349.

Just merged

@nickvergessen
Copy link
Member

Taking over since kessel is sick

@nickvergessen nickvergessen force-pushed the 3rdparty-drop-php-parser branch from 2a875e2 to 26b34fe Compare February 2, 2023 11:08
Signed-off-by: Daniel Kesselberg <[email protected]>
@nickvergessen nickvergessen force-pushed the 3rdparty-drop-php-parser branch from 26b34fe to d923565 Compare February 2, 2023 11:42
@nickvergessen
Copy link
Member

Repushed to have --no-dev committed nextcloud/3rdparty#1303

@nickvergessen nickvergessen added this to the Nextcloud 26 milestone Feb 2, 2023
@nickvergessen nickvergessen added 3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update labels Feb 2, 2023
@nickvergessen
Copy link
Member

Update documentation

@kesselb please take care of this.

@nickvergessen nickvergessen merged commit e4f0505 into master Feb 2, 2023
@nickvergessen nickvergessen deleted the 3rdparty-drop-php-parser branch February 2, 2023 13:25
@kesselb kesselb removed the pending documentation This pull request needs an associated documentation update label Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants