pharmaverse / admiral

ADaM in R Asset Library
https://pharmaverse.github.io/admiral
Apache License 2.0
220 stars 62 forks source link

Feature Request: Further customise of our .lintr file #2476

Closed manciniedoardo closed 2 months ago

manciniedoardo commented 2 months ago

Feature Idea

What if we leveraged the functionality in {lintr} to make custom linters?

One example I can think of is related to our recent error messaging update. Now that we use {cli}, do we ever need to use stop() or warning()? If not, we could make a custom linter for them to avoid ever introducing them in our codebase.

@pharmaverse/admiral any thoughts?

Some thoughts:

bms63 commented 2 months ago

I'm curious how we can make this work for us especially since we are reaching more of a steady state. This way some of our programming conventions can be automatically checked and enforced.

.... but I feel like it should be a discussion rather than an issue.