Message270899
Mathieu: Maybe you haven’t seen some of the comments on your older patches. E.g. my comment on fromisoformat4.patch about improper use of “with self.assertRaises(...)” still stands.
Also, adding some documentation to the patch might help the likes of Anders figure out the scope of the change. I think we decided to parse RFC 3339’s “internet date and time format” profile of ISO 8601 with the date, time, and datetime classes, including tolerating arbitrary resolutions of fractions of seconds in the time, and parsing time zones.
I don’t think we need to test every combination of the other ISO 8601 formats. There are already a couple of negative tests. Are there any in particular you think are important to add? |
|
Date |
User |
Action |
Args |
2016-07-21 01:21:35 | martin.panter | set | recipients:
+ martin.panter, barry, jcea, roysmith, belopolsky, nagle, vstinner, jwilk, mcepl, eric.araujo, Arfrever, r.david.murray, davydov, cvrebert, karlcow, SilentGhost, perey, flying sheep, mihaic, aymeric.augustin, Roman.Evstifeev, berker.peksag, piotr.dobrogost, kirpit, Anders.Hovmöller, jstasiak, Eric.Hanchrow, deronnax, pbryan |
2016-07-21 01:21:35 | martin.panter | set | messageid: <1469064095.34.0.292073932617.issue15873@psf.upfronthosting.co.za> |
2016-07-21 01:21:35 | martin.panter | link | issue15873 messages |
2016-07-21 01:21:34 | martin.panter | create | |
|