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

Qual: Fix notices, update baseline, enable PhanTypeMismatchArgument #32901

Merged
merged 4 commits into from
Feb 4, 2025

Conversation

mdeweerd
Copy link
Contributor

@mdeweerd mdeweerd commented Feb 1, 2025

Qual: Fix notices, update baseline, enable PhanTypeMismatchArgument

  • Fix phan notices mostly in relation with supplier invoices,
  • Fix phan notices in relation with formquestion (phpdoc update, casts).Update the baseline for fixes.
  • Enable PhanTypeMismatchArgument as this is being fixed and to avoid
    that new cases occur in files where these are fixed.
    Only about 3800 cases remain, was more than 12000 when starting with Phan.

@mdeweerd mdeweerd force-pushed the qual/phan.2025.1.31 branch 3 times, most recently from 2b0feab to d6c3622 Compare February 1, 2025 15:38
@mdeweerd
Copy link
Contributor Author

mdeweerd commented Feb 1, 2025

Note: this PR will be split in 2 and is based on #32842 so will convert this as "Ready" when that is integrated.
In practice this PR is ready.

@mdeweerd mdeweerd force-pushed the qual/phan.2025.1.31 branch 4 times, most recently from 4acd524 to 0c7d6b2 Compare February 2, 2025 04:10
@mdeweerd mdeweerd marked this pull request as ready for review February 2, 2025 04:14
@mdeweerd mdeweerd force-pushed the qual/phan.2025.1.31 branch from 0c7d6b2 to 952e033 Compare February 2, 2025 13:50
# Qual: Fix phan notices

Fix phan notices in fourn/commande (+form class)
# Qual: Fix phan notices

Fix phan notices mostly in relation with supplier invoices,
Fix phan notices in relation with formquestion (phpdoc update, casts).
# Qual: Update baseline, enable PhanTypeMismatchArgument

Update the baseline for fixes.
Enable PhanTypeMismatchArgument as this is being fixed and to avoid
that new cases occur in files where these are fixed.
Only about 3800 cases remain, was more than 12000 when starting with Phan.
@mdeweerd mdeweerd force-pushed the qual/phan.2025.1.31 branch from 952e033 to f750f2c Compare February 4, 2025 01:09
@eldy eldy merged commit 52bbbcd into Dolibarr:develop Feb 4, 2025
6 checks passed
@mdeweerd mdeweerd deleted the qual/phan.2025.1.31 branch February 4, 2025 23:17
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.

2 participants