Issue #821 - Remove digits argument from get_correlations

[nikosbosse]

Description

This PR closes #821.

This PR moves digits argument from get_correlations to plot_correlations. Removing the digits was discussed in #821. I think it makes sense to move it to plot_correlations() - ultimately, I can't really imagine a case in which one would like to plot correlations without rounding them.

Checklist

• [x] My PR is based on a package issue and I have explicitly linked it.
• [x] I have included the target issue or issues in the PR title as follows: issue-number: PR title
• [x] I have tested my changes locally.
• [x] I have added or updated unit tests where necessary.
• [x] I have updated the documentation if required.
• [x] I have built the package locally and run rebuilt docs using roxygen2.
• [x] My code follows the established coding standards and I have run lintr::lint_package() to check for style issues introduced by my changes.
• [ ] I have added a news item linked to this PR.
• [ ] I have reviewed CI checks for this PR and addressed them as far as I am able.

[nikosbosse]

I just checked and we never had that argument anywhere specifically. The old correlation function just did it by default...

[nikosbosse]

Added a news item now

[jamesmbaazam]

I just checked and we never had that argument anywhere specifically. The old correlation function just did it by default...

That's great; no deprecation is needed.