search

teemtee / tmt

Test Management Tool
MIT License
85 stars 129 forks source link

Refactor the `DiscoverFmf.go()` method #2448

Open psss opened 1 year ago

psss commented 1 year ago

It has grown just too much (almost 400 lines) and it's very hard to read it. Let's split it into multiple methods.

thrix commented 10 months ago

@bajertom no PR yet opened, are you making it please?

bajertom commented 10 months ago

@thrix Hello, I had that planned for this weekend, however I see Lukas's proposed changes in the draft and it seems they would would kinda overlap. Should I proceed or wait for the draft to be implemented?

lukaszachy commented 8 months ago

During the refactor https://github.com/teemtee/tmt/issues/2737 might get fixed, or not. Linking here for awareness that current behavior might be buggy.