Commit 86062fe
fix(migrated PR 4207): issue 4206 (#4279)
* Initial fix in interp_star_status function
* Removed junk lines and fixed a typo
* ci: auto fixes from pre-commit.com hooks.
for more information, see https://pre-commit.ci
* chore: adding changelog file 4279.fixed.md [dependabot-skip]
* chore: adding changelog file 4279.fixed.md [dependabot-skip]
* style: add type annotations to parameter parsing helpers in parameters.py
Add explicit type hints for math_header, array_header, is_parameter_listing,
is_math_listing, is_array_listing, and interp_star_status to improve clarity
and support static analysis.
* test: add tests for C_FullFile parameter handling in interp_star_status
Add golden "c_fullfile status" sample and multiple tests to verify
interp_star_status correctly parses C_FullFile entries. Cover mixed
SMAT/C_FullFile outputs, single C_FullFile outputs, and various
C_FullFile-like names; ensure only the "type" field is present for
C_FullFile parameters and other fields (value, MemoryMB, workspace,
shape) are not included. Also include the c_fullfile case in the
parameterized status test.
---------
Co-authored-by: Chai Fillerup <[email protected]>
Co-authored-by: ChaiF1 <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Camille Latapie <[email protected]>
Co-authored-by: German <[email protected]>1 parent 282883d commit 86062fe
1 file changed
+1
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
0 commit comments