-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP Scilpy bot v2 #1033
base: master
Are you sure you want to change the base?
WIP Scilpy bot v2 #1033
Conversation
…se between the two
…BUG to display the whole docstring
Hello @frheault, Thank you for updating !
Comment last updated at 2024-10-11 15:14:13 UTC |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1033 +/- ##
==========================================
+ Coverage 68.84% 68.88% +0.03%
==========================================
Files 431 432 +1
Lines 22332 22517 +185
Branches 3332 3049 -283
==========================================
+ Hits 15375 15511 +136
- Misses 5670 5709 +39
- Partials 1287 1297 +10
|
@mdesco Could you test this and imagine being Laurent / Sami trying to find a script (or information). Is it better than before? Is it intuitive? |
Quick description
This version has some duplicated code removed, some edge cases fixed, highlighting now keeps the formatting.
This must be tested, also the vocabulary file could be cleaned and improved during a hackathon (when is the next one?)
...
Type of change
Check the relevant options.
Provide data, screenshots, command line to test (if relevant)
...
Checklist