cqfn / pdd

Command line toolkit for collecting TODO markers from your code, known as Puzzle Driven Development (PDD)
https://www.0pdd.com
MIT License
76 stars 25 forks source link

[#201] add `--model` flag to mark puzzles for prioritization using 0pdd #202

Closed mbao01 closed 2 years ago

mbao01 commented 2 years ago
mbao01 commented 2 years ago

@yegor256 please review

yegor256 commented 2 years ago

@mbao01 the description of this feature is not clear (at least to me). How exactly will it work? Let's make it more obvious in README

codecov[bot] commented 2 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@261c2c3). Click here to learn what that means. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #202   +/-   ##
=========================================
  Coverage          ?   95.50%           
=========================================
  Files             ?       10           
  Lines             ?      356           
  Branches          ?        0           
=========================================
  Hits              ?      340           
  Misses            ?       16           
  Partials          ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

mbao01 commented 2 years ago

@yegor256 please review

yegor256 commented 2 years ago

@mbao01 I don't understand why we need this feature here. It belongs only to 0pdd. We can add this "marker" to XML when we process it over there. Here, in the command line, why do we need to care about ML part of 0pdd project??

mbao01 commented 2 years ago

@yegor256 I believe this feature is to allow each user (or project) to opt-in to use the ML model. I was unaware that they have access to configure 0pdd. I see that now in the 0pdd readme and will move this feature there.