Missing `@sort_by_option` decorator for `SortByOption`

[thibgc]

The docstring of the SortByOption class (in 7.18.0) alludes to a sort_by_option decorator:

        @command
        @table_format_option
        @sort_by_option(
            ("Package ID", "package_id"),
            ("Name", "package_name"),
            ("Manager", "manager_id"),
            ("Version", None),
        )
        @pass_context
        def my_cmd(ctx):
            ctx.print_table(header_defs, rows)

but it doesn't seem to exist. Could this be added so that the sort option could be grouped with other options or be part of constraints?

[kdeldycke]

Good catch! Probably something I missed in my original implementation of SortByOption from d5cc54d in v7.11.0. Your instinct is good as you can see my pattern of having a ready to use pre-configured decorator for each of my custom option classes in Click Extra. So the same way TableFormatOption has a @table_format_option, SortByOption should have a @sort_by_option. Let me fix that...

[kdeldycke]

BTW, looks like you are relying on 7.18.0 which is good practice to depend on an aged release to prevent supply chain attack. 😁 I am currently iterating fast on Click Extra as I am breaking and cleaning up a lot of stuff for a major 8.0.0 that was just released yesterday. So 2 strategies here: you are not in a hurry and keep waiting patiently for the 8.x series to mature a bit before upgrading, or you jump straight to 8.0.0 today and help me identify the last edge-cases (in which case you can expect fixes you pointed out to be release fast).