Conversation
ansible_community_meetings.ics
Outdated
@@ -8,11 +8,11 @@ DURATION:PT1H | |||
RRULE:FREQ=WEEKLY | |||
DESCRIPTION:Project: Ansible Security Automation working group\nChair: A | |||
dam Miller (maxamillion)\, Sumit Jaiswal (justjais)\, Abhijeet Kasurde (ak | |||
asurde)\nDescription: \nWe discuss everything about Information Security | |||
asurde)\nDescription: \nWe discuss everything about Information Security |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder whether these trailing spaces are important and need to stay. Looking at the other lines which don't have spaces, and where words are just broken by a line end, I think they need to stay.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you're right. Maybe adding spaces to the beginning of the lines will be easier to maintain as we will know they're intentional (and IDEs are less likely to remove those automatically).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oof, my editor must have "done me a favor" there - I didn't intend to change the whitespace. Let me fix that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abadger I thought that this file was auto-generated anyway from the YAML file, so usually nobody should edit it manually.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, you are correct.
9e5f2d3
to
b6759e1
Compare
Note to Future Self: next time update the yaml manually, run make in the meetings subdirectory to update the ics files, then check everything in. |
merged, thanks! |
Per our discussion on March 23, changing the UTC time to one hour earlier for the summer. Depending on your local time, this may mean next week's meeting is at the same local time, or one hour earlier local time.
Related to #579 and to #576.