Skip to content
Snippets Groups Projects
  1. May 12, 2016
    • ulif's avatar
      Remove overcomplicated stuff from randomsource. · 0451f247
      ulif authored
      While it is nice to have a working plugin example, it looks also a bit
      overcomplicated to me. It is better for contributors, if they can find
      things (like CLI argument parsing) in one (foreseeable) place and do
      not have for things scattered over the whole project.
      
      For plugin documentation we can use the sphinx docs. Or maybe provide
      a sample plugin in tests.
      0451f247
  2. May 11, 2016
    • ulif's avatar
      Add `--dice-sides` option. · 6bfdbd2e
      ulif authored
      This argument makes sense together with ``realdice`` randomsource
      and tells how many sides the dice we use have. Six by default.
      6bfdbd2e
    • ulif's avatar
      Use dict in options test. · b2aada6a
      ulif authored
      As we will start to do real things with the options passed in, we need
      a real (options) dict also in other tests of `RealDiceRandomSource`.
      b2aada6a
    • ulif's avatar
      Add a (dummy) argparser updater for real dice. · 44aa926c
      ulif authored
      The RealDiceRandomSource will support own commandline arguments. This
      is handled by an own classmethod called `update_argparser` that is
      called upon initialization of diceware.
      44aa926c
    • ulif's avatar
      Update docs. · f90da2ec
      ulif authored
      f90da2ec
    • ulif's avatar
      pep8. · 7ff2d0c1
      ulif authored
      7ff2d0c1
    • ulif's avatar
      Update docs. · 37b21f14
      ulif authored
      37b21f14
    • ulif's avatar
      Allow plugins to work with the argparser. · 03fa1a63
      ulif authored
      If plugins want to work with their own commandline args, we allow
      this by calling their `update_argparser` methods. This has to be a
      classmethod, if it exists.
      03fa1a63
    • ulif's avatar
      Next release will be more than pure bugfix. · 29f16960
      ulif authored
      We got API changes, new options, etc. All this will make the
      next release a minor one, not a pure bugfix.
      29f16960
  3. May 10, 2016
  4. May 09, 2016
  5. May 08, 2016
  6. May 07, 2016
  7. May 06, 2016
  8. May 05, 2016
    • dwcoder's avatar
      Fix issue 23 · 5ca58b56
      dwcoder authored
      This commit fixes #23
      
      Add the `string.strip()` function to the `get_config_dict()`, to remove
      inverted commas around string values in the diceware.ini file.
      The strip removes both `"` and `'`.
      Add unit test to ensure the line:
      
          delimiter=" "
      
      in the diceware.ini file is being parsed correctly as a space.
      5ca58b56
  9. May 03, 2016
  10. May 01, 2016
  11. Apr 30, 2016
    • ulif's avatar
      Update history. · 88f049d9
      ulif authored
      88f049d9
    • ulif's avatar
      More kudos to @dwcoder · 0fbe13f4
      ulif authored
      0fbe13f4
    • ulif's avatar
      Merge branch 'dwcoder-fix-iss21' · ceefc71e
      ulif authored
      ceefc71e
    • ulif's avatar
      Undo changes to "considers_configfile" tests. · 3cbbc449
      ulif authored
      The modifications undone here, in
      `test_handle_options_considers_configfile()` could have gone into an
      own test. Each single test should only test a clear minimum of
      things.
      
      This specific test checked, whether configiles are considered at
      all. It was not its purpose to examine correct interpretation of
      keywords in the .ini-file; only to see whether these keyword values
      can be changed at all.
      
      If we wanted a complete and strict checking also of values, we had at
      least to check all possible keywords, including 'randomsource' and
      'wordlist'. Instead we only check a boolean, int, and string type
      keyword.
      
      Therefore the change to `test_handle_options_considers_configfile()`
      should have gone to another test.
      
      But then, the added testing is not neccessary. We had a test already
      that should have checked the 'caps' value properly. This test was done
      wrong before and is now done correctly, thanks to @dwcoder .
      
      I therefore removed the added test functionality.
      3cbbc449
Loading