YARD/MeaninglessTag: Allow @param on Data/Struct defintions

[lekemula]

Hi,

The following Data/Struct annotations, are not meaningless anymore from solargraph's perspective (see castwide/solargraph#970 and castwide/solargraph#939):

# @param required_gems [Array<String>]
# @param helper_modules [Array<String>]
GemHelpers = Struct.new(:required_gems, :helper_modules, keyword_init: true)

Would it make sense to exclude such Struct/Data definitions via const assignments or at least add a configuration option to the YARD/MeaninglessTag cop?

I can draft a PR if you find this acceptable.

[ksss]

@lekemula Thank you for your proposal.
I understand the situation.
I think both are acceptable proposals.

[lekemula]

Hi @ksss, I'm having trouble finding a way to test my changes that I intend to make. As you appear to be using an unconventional approach to testing the cops without using specs/tests, could you explain what the typical development workflow looks like?

I understand how the smoke tests are set up more or less, but I can't attach a debugger. I tried all sorts of ways of attaching it, but even this didn't seem to help:

rdbg -c -- bundle exec rubocop --require ./lib/rubocop-yard.rb --only YARD/MeaninglessTag --debug smoke/meaningless_tag_data_struct_definitions.rb

Thanks for your help.

[ksss]

I honestly regret the testing method...

In my environment, the debugger launched as follows.

$ bundle exec rdbg -c -- bundle exec rubocop --debug --only YARD/MeaninglessTag smoke/meaningless_tag.rb
...snip...
(rdbg) b RuboCop::Cop::YARD::MeaninglessTag#check
Unknown constant name: "Cop"
#0  BP - Method (pending)  RuboCop::Cop::YARD::MeaninglessTag#check
(rdbg) c
Inspecting 1 file
[30, 39] in ~/src/github.com/ksss/rubocop-yard/lib/rubocop/cop/yard/meaningless_tag.rb
    30|         end
    31|         alias on_module on_class
    32|         alias on_casgn on_class
    33|
    34|         def check(node)
=>  35|           preceding_lines = preceding_lines(node)
    36|           return false unless preceding_comment?(node, preceding_lines.last)
    37|
    38|           docstring = build_docstring(preceding_lines)
    39|           return false unless docstring