Skip to content

calcurse-holiday to create holiday events for irregular dates#357

Open
atticus-sullivan wants to merge 4 commits intolfos:pufrom
atticus-sullivan:holiday
Open

calcurse-holiday to create holiday events for irregular dates#357
atticus-sullivan wants to merge 4 commits intolfos:pufrom
atticus-sullivan:holiday

Conversation

@atticus-sullivan
Copy link

Most things should be stated in the first commit and in the README.md

As I'm new in this thing, I've got a question:

  • Is this the right branch on which I've created the pull request? (or should this have been the master branch? Maybe this should be clarified in the contributing section of the project README.md)
  • Please tell me what the next steps are (this is my first pull request to an open source project)
  • I have no experience with automake if steps are required in this area it would be very kind if you could give hints on what to do.

I've tried to write clean code, but if I've violated some guidelines you have please tell me.

Notes:

  • I've seen you normally use configparser for the config files (which in my case would be the holiday.yaml file). I've used yaml since I know this and since it's pretty easy to have lists to cycle one.

For the sake of uniform configs and to reduce dependendies, I'm willing to switch. I'd just be interested in how you'd realize these sort of config with configparser (I pretty much like the ability that keys can have whitespaces in yaml...)

  • I've seen in the caldav "plugin" textwrap is used. Do you think this would be appropriate here too for the output on the console?

For the functionality see `-h` and the 'README.md'.
I think this is useful since it fills the gap for creating events for
holiday days which dates are based on quite complicated functions like
Eastesunday.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant