-
Notifications
You must be signed in to change notification settings - Fork 26
Tidy: use internal concatenation to build error #102
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
Valid point. [ Also note that the package is pre-Rcpp and uses .Call() everywhere. Changing that would be nice. "Bigger project still. " ] Maybe it is time we hand this package over to you guys, and/or you start maintaining an internal fork. |
I can probably merge it if you please add a ChangeLog entry. |
I'm afraid you're still (by a pretty wide margin?) the person with the best understanding of the code base & eng support is unfortunately lacking on our end :( But do feel free to assign issues to us (me 😄) & if you're dying to pass off maintainership, please flag & I'll see what we can do. |
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.
Thanks!
Glad to see you guys are still using this at Google. =) We used it at
Apple some but it wasn't as useful as without the global protodb in google3
(or whatever the equivalent is now, ten years later).
- Murray
…On Tue, May 28, 2024 at 1:55 PM Michael Chirico ***@***.***> wrote:
Maybe it is time we hand this package over to you guys, and/or you start
maintaining an internal fork.
I'm afraid you're still (by a pretty wide margin?) the person with the
best understanding of the code base & eng support is unfortunately lacking
on our end :(
But do feel free to assign issues to us (me 😄) & if you're dying to pass
off maintainership, please flag & I'll see what we can do.
—
Reply to this email directly, view it on GitHub
<#102 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGC53L62KRWE2GDOTW7HLTZETVLVAVCNFSM6AAAAABINTFKQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZWGA4DQNZZHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
You are way overestimating my familiarity with the code base, but I do still know which buttons to press to keep it in good standing at CRAN. The effective modus operandi here where all changes come from an unnamed large tech company which may be deploying this internally works for now. |
Nice seeing you Murray! More used to seeing your signature on blames deep in R infra here :) Global DB lives on 🤘 |
As found by
condition_message_linter()
.We might also prefer
gettextf()
for internationalization but that's a much bigger project.