Replies: 1 comment 1 reply
-
Thanks, should be fixed in 075c84d.
I hope you're recovering well. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Help
Description
The new "balanced" reporter looks great. Just giving some feedback that might help people transitioning...
I just upgraded to 1.11.3 and was puzzled by the warning message
The summary reporter in tar_make() etc. was deprecated in targets version 1.10.1.91010 (2025-03-31). Use the "balanced", "verbose", or "timestamp" reporter instead.
My puzzlement was because I was using "timestamp_positives" not "summary" - it took me a little bit to get my head around the changes (it's my first day back after surgery - the brain's not necessarily 100% 🙃) and some experimentation to discover that "timestamp" now doesn't report individual skipped targets.
Perhaps "timestamp_positives" could be aliased to "timestamp" to ease transitioning (with a warning about deprecation and noting that it has the same behaviour?). Presumably the same thing would hold with "verbose_positives" wrt "verbose".
I hadn't used other deprecated reporters, but if they could similarly benefit from a more specific message it might help others who land in that "the thing I was doing stopped working, how do I get it doing what I want?" puddle.
Beta Was this translation helpful? Give feedback.
All reactions