SNEWS2 / SNEWS_Publishing_Tools

Publishing Tool for SNEWS
BSD 3-Clause "New" or "Revised" License
2 stars 2 forks source link

Iso time #52

Closed joesmolsky closed 2 years ago

joesmolsky commented 2 years ago

Change time format to ISO format. This makes the TimeStuff class obsolete. Times can now be set with datetime.utcnow().isoformat().

Version also changes from 1.1.0 to 1.1.1

KaraMelih commented 2 years ago

Your commits from yesterday started to fail the tests; https://github.com/SNEWS2/SNEWS_Publishing_Tools/actions could you have a look at the cause

joesmolsky commented 2 years ago

I have updated the format checker to include warnings with the logs.

Also did some fixes to the time checking logic and fixed some typos in the test function times.

KaraMelih commented 2 years ago

Thanks Joe it looks better! I can try to test it tomorrow, latest next Monday and merge it. Please let me know if you need me to do it faster

KaraMelih commented 2 years ago

You can also debug these via snews_pt run-scenarios it shows already these error messages without having to interact with cs