I haven't had time to review this properly, but as a reminder to myself I need to do some real checking that the resulting iCalendar is correct. The obvious gotcha here is at least the BYWEEKDAY, which is just a dateutil convention :
"Notice that in the RFC documentation, this is specified as BYDAY, but was renamed to avoid the ambiguity of that keyword. "
There might be more, but likely the more tricky parts are in the public interface patch.
I haven't had time to review this properly, but as a reminder to myself I need to do some real checking that the resulting iCalendar is correct. The obvious gotcha here is at least the BYWEEKDAY, which is just a dateutil convention :
"Notice that in the RFC documentation, this is specified as BYDAY, but was renamed to avoid the ambiguity of that keyword. "
There might be more, but likely the more tricky parts are in the public interface patch.