Use Option class for defining options

[Ryan J Ollos]

Cool plugin! I have a minor suggested improvement, which can be found in the attached patch.

If you use the Option class, and provide documentation, it will display on the TracIni page:

The patch also fixes a few ​PEP8 violations.

[Ryan J Ollos]

Btw, the patch is not quite complete. The placeholders Put some documentation here need to be replaced with real documentation.

[Graham Miln]

Thank you for the patch; I appreciate your looking over the code.

Fixed in r13008.

[Ryan J Ollos]

Sure, no problem. I might have a few other suggestions related to how the markup strings are generated. I'll see if I can find time to take another look.

Have you been working with Trac and Python for long? I'm always curious as to how people are using Trac, particularly plugin authors.

Hint: You can use ​t:CommitTicketUpdater to have commit messages inserted into ticket. For example, add Refs #11036 or Fixes #11036 in your commit message. You can see some examples in various tickets, such as #10218.

[Graham Miln]

Thank you. I have responded via e-mail to your questions.