DimitrisJim

Results 111 comments of DimitrisJim

but I don't think this is such a good first issue anymore, I'll try and re-structure the cli argument parsing to make it easier to generate these.

@yugborana great! You basically need to update the workflow file to add the target. If you take a look at the current workflow, there's already some checks made [for other...

don't think issue is relevant actually, looking at the file the user was referring to, the unrecognized token is just a large triple quoted string.

I think something along these lines must be tested in `test_ordered_dict.py`. There's a class that runs some of the ordered dict test functions on the `dict` itself: https://github.com/RustPython/RustPython/blob/fa790558211ec690541299258f44d7453501958a/Lib/test/test_ordered_dict.py#L749-L765

> Behaviors on CPython: work normally this is an overflow in CPython and should similarly be one in RustPython. The change should be relatively straight-forward: when trying to convert the...

> So a possible solution is to write the logic that would increment everything to be valid inputs for Chronos from_ymd_opt and from_hms_opt ugh, that's unfortunate but thanks for the...

Open an issue on CPython, I'd say. IMHO this should at least be documented if other implementations are to follow or the tests need to be separated (and calls that...

this is a Python level error so first place I'd look is if maybe `email` is not up to date?

both `tm_gmtoff` and `tm_zone` are missing from the `struct_time` struct: https://github.com/RustPython/RustPython/blob/59df92d700fdf479b7294d44ff3f340ec9a0769c/vm/src/stdlib/time.rs#L318-L328 reference, Python docs on these: https://docs.python.org/3/library/time.html#time.struct_time Might be easy to add them.

Should be duplicate of https://github.com/RustPython/RustPython/issues/4613 since this also appears to fail due to the `format(123456789, "n")` call.