-
Notifications
You must be signed in to change notification settings - Fork 37
Joint dependency updates and Claude improvement recommendations #516
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
Conversation
* pushing this for now while I make a PR for TimeSeries.jl to update their PrettyTables compat, since TimeSeries is fixing PrettyTables to less than 3.0.0 right now * updated PrettyTables and all tests passing
Project.toml
Outdated
| Mustache = "1" | ||
| Pkg = "1" | ||
| PrettyTables = "1.3 - 3" | ||
| PrettyTables = "2.1, 3.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jd-lara is this how I should implement the PrettyTables version for all the repos, or is this a special case for IS?
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (45.32%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #516 +/- ##
==========================================
- Coverage 79.10% 78.42% -0.69%
==========================================
Files 71 73 +2
Lines 6192 6247 +55
==========================================
+ Hits 4898 4899 +1
- Misses 1294 1348 +54
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
This PR follows the same approach that DataFrames.jl had for backwards compatibility of PrettyTables and avoid a major release