Conversation
TobiasKadelka
force-pushed
the
helptext
branch
4 times, most recently
from
fa53edd to
0603e1c
Compare
aqw
left a comment
aqw
left a comment
There was a problem hiding this comment.
Cool stuff! I left some comments. :-)
TobiasKadelka
force-pushed
the
helptext
branch
2 times, most recently
from
eff353a to
a0d6d93
Compare
|
@aqw Thanks for your feedback! I went through it and adjusted the help texts :) |
aqw
left a comment
aqw
left a comment
There was a problem hiding this comment.
Looks way better.
I left a few nits. Overall getting close.
I'm also interested in @bpoldrack's thoughts about tree.
|
Everything was updated and pushed :) |
|
Please fix the |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #790 +/- ##
=======================================
Coverage ? 94.45%
=======================================
Files ? 81
Lines ? 8403
Branches ? 822
=======================================
Hits ? 7937
Misses ? 349
Partials ? 117 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I did fix the ruff link failures and now I think this PR is ready to merge :-D |
3 participants
Works on #789
I did adjust some existing text because I felt like these sentences can be phrased in a way that is easier and more intuitive to parse for the broad audience. If there is disagreement about these changes, I welcome discussions and can obviously revert parts of it :)