sventec

icon location Lawrence Berkeley Lab icon location not Sventek

Results 4 comments of sventec

Add PDM to Package Management

The PDM docs URL has since changed to (though the current link in this PR still redirects to the new one). +1 for merge, I switched from Poetry for PEP...

requests_common.py should not raise an error when HTTP response code is greater than 299

Echoing this, it would be beneficial to support returning results from `RequestsCommon`'s `execute` (and thus `execute_call_v2`) with a non-2XX status code in some cases. I experienced a similar issue to...

requests_common.py should not raise an error when HTTP response code is greater than 299

This may be better handled by implementing a `callback` function in the `fn_utilities` `Utilities: Call REST API` function, if there are no other situations where _not_ raising a status code...

requests_common.py should not raise an error when HTTP response code is greater than 299

@mscherfling thank you for the callout on `fn_rest_api`. The improvement you described would exactly satisfy the need expressed in my earlier comment. Will move to migrate and implement via `rest_api_allowed_status_codes`...