metafacture-core icon indicating copy to clipboard operation
metafacture-core copied to clipboard

Improve Flux HelpPrinter content

Open fsteeg opened this issue 3 years ago • 5 comments

This is part of https://github.com/metafacture/metafacture.github.io/issues/10

Improve content of HelpPrinter.java:

  • [ ] Group commands by type (Readers, Decoders, Encoders, Writers)
  • [ ] Explain signatures (a general explanation about how to read signatures like String -> StreamReceiver or specific translations like "Reads from a string and writes to a stream receiver" or "Expects strings as input and produces metadata events")
  • [ ] Add missing options

fsteeg avatar Jun 01 '21 13:06 fsteeg

It would be nice if you link to the code of a function too .

TobiasNx avatar Jul 14 '21 11:07 TobiasNx

Could you elaborate a little?

Group commands by type

What is a command's type?

Explain signatures

What might such an explanation look like?

Add missing options

Which options are missing?

I would maybe add:

  • [ ] Sort options alphabetically
  • [ ] List signature before options
  • [ ] Add (mandatory and optional) arguments via Class.getConstructors() (instead of @ReturnsAvailableArguments annotation?)

And as a slight variation of @TobiasNx's suggestion:

  • [ ] Link to class's Javadoc
  • [ ] Maybe even: Link to option's (i.e., setter's) Javadoc

blackwinter avatar Oct 13 '21 20:10 blackwinter

Could you elaborate a little?

I've added some details to the original issue description.

fsteeg avatar Oct 14 '21 07:10 fsteeg

@dr0i: You checked the task for "Link to class's Javadoc" in my comment, but the recent changes (#509) only link to the source code, not the Javadoc.

blackwinter avatar Nov 27 '23 16:11 blackwinter

@blackwinter oh, right - I thought of the javadoc comments in the classes...

dr0i avatar Nov 28 '23 08:11 dr0i