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.

Sign up for GitHub

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

Jump to bottom

phpstan level 7 for importExportController.php #5361

Merged
merged 15 commits into from May 4, 2023
Merged

phpstan level 7 for importExportController.php #5361

merged 15 commits into from May 4, 2023

Conversation

ColonelMoutarde
Copy link
Contributor

Changes proposed in this pull request:

  • phpstan level 7 for importExportController.php

Pull request checklist:

  • clear commit messages
  • code manually tested

@Alkarex Alkarex added this to the 1.22.0 milestone Apr 30, 2023
@Alkarex Alkarex added the System care Everything related to the system care label Apr 30, 2023
ColonelMoutarde and others added 3 commits May 1, 2023 20:25
@ColonelMoutarde @Alkarex
Update app/Controllers/importExportController.php
acb8457 
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
@ColonelMoutarde @Alkarex
Update app/Controllers/importExportController.php
fb11560 
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
@ColonelMoutarde @Alkarex
Update app/Controllers/importExportController.php
fe3a402 
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
@ColonelMoutarde @Alkarex
Update app/Controllers/importExportController.php
fa35df4 
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
@ColonelMoutarde ColonelMoutarde deleted the feature/phpstan-7-for-importexportcontroller branch May 3, 2023 08:38
@ColonelMoutarde ColonelMoutarde restored the feature/phpstan-7-for-importexportcontroller branch May 3, 2023 08:44
@Alkarex
Copy link
Member

Alkarex commented May 4, 2023

I have made a few edits. You are welcome to double-check d04aaa9

@ColonelMoutarde
Copy link
Contributor Author

All right for me

@Alkarex Alkarex merged commit aa30635 into FreshRSS:edge May 4, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Alkarex Alkarex Alkarex left review comments

Assignees
No one assigned
Labels
System care Everything related to the system care
Projects
None yet
Milestone
1.22.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ColonelMoutarde @Alkarex