semester_first_monday command line argument

[Nikita-Boyarskikh]

• You should set default value of semester_first_monday smarter or not set it at all
• Type of it should be date_parser instead of str Like this:

  def date_parser(date):
  try:
      format = '%d-%m-%Y'
      return datetime.strptime(date, format)
  except ValueError:
      msg = 'date has incorrect format. It must have format: `dd-mm-yyyy`'
      raise argparse.ArgumentTypeError(msg)

[gabolaev]

1) There is a way to parse it from somewhere, but from where?

[gabolaev]

2. Maybe it's better to set help info of how user should pass semester_first_monday. Like "set it please as dd-mm-yyyy"

[stek29]

zaebali viebivatsya, pishite ra russkom

[Nikita-Boyarskikh]

@gabolaev, 2. one does not exclude the other :)

[gabolaev]

@Nikita-Boyarskikh yeah, i meant the thing you add in "edited" :)

[gabolaev]

@Nikita-Boyarskikh i'll add it soon, thanks

[Nikita-Boyarskikh]

@gabolaev, in "edited" I wrote a version of date parser. It does not exclude one more line about it in help message

[Nikita-Boyarskikh]

@gabolaev, see https://docs.python.org/3/library/argparse.html#type

[gabolaev]

thanx