Skip to content

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

Merged
merged 2 commits into from
May 28, 2024

Conversation

Image for: Conversation
Copy link
Contributor

As found by condition_message_linter().

We might also prefer gettextf() for internationalization but that's a much bigger project.

Copy link
Owner

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.

Copy link
Owner

I can probably merge it if you please add a ChangeLog entry.

Copy link
Contributor Author

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.

Copy link
Owner

eddelbuettel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Collaborator

murraystokely commented May 28, 2024 via email

Copy link
Owner

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.

eddelbuettel merged commit 267e1f0 into eddelbuettel:master May 28, 2024
2 checks passed
MichaelChirico deleted the patch-1 branch May 28, 2024 21:10
Copy link
Contributor Author

MichaelChirico commented May 28, 2024

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

Nice seeing you Murray! More used to seeing your signature on blames deep in R infra here :)

Global DB lives on 🤘

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants