fix: replace deprecated parseAll with parse_all in interpolate.py

[ritoban23]

Fixes #10968

Description

Replaced the deprecated parseAll parameter with the PEP8-compliant parse_all parameter in the parseString method call.

Changes

• Updated line 181 in dvc/parsing/interpolate.py to use parse_all=True instead of parseAll=True

• ❗ I have followed the Contributing to DVC checklist.

• 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.

[CLAassistant]

All committers have signed the CLA.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.92%. Comparing base (2431ec6) to head (5f2a71a).
⚠️ Report is 187 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10971      +/-   ##
==========================================
+ Coverage   90.68%   90.92%   +0.23%     
==========================================
  Files         504      505       +1     
  Lines       39795    41084    +1289     
  Branches     3141     3249     +108     
==========================================
+ Hits        36087    37354    +1267     
- Misses       3042     3083      +41     
+ Partials      666      647      -19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[skshetry]

Have you tested this locally? Also, does this PR work with this constraint we have in pyproject.toml?

dvc/pyproject.toml

Line 67 in 04cbd40

"pyparsing>=2.4.7",

[ritoban23]

@skshetry Thanks for pointing it out...Just checked the change won't work with current pyparsing>=2.4.7 constraint since parse_all wasn't introduced until pyparsing 3.0.0. We'd need to either bump the constraint or keep using the old parseAll parameter.

[skshetry]

@skshetry Thanks for pointing it out...Just checked the change won't work with current pyparsing>=2.4.7 constraint since parse_all wasn't introduced until pyparsing 3.0.0. We'd need to either bump the constraint or keep using the old parseAll parameter.

Since that was released more than 4 years ago, I think it's okay to update the constraint to >=3.0.0.

Also, did you check before and after this PR on whether DeprecationWarning gets raised or not?

[ritoban23]

Also, did you check before and after this PR on whether DeprecationWarning gets raised or not?

Yes checked for the deprecation warning

[skshetry]

Thank you. 🙂