Skip to content

MINOR: [R] Fix a package namespace warning#43737

Merged
jonkeane merged 3 commits intoapache:mainfrom
jonkeane:fix_dt_warn
Aug 17, 2024
Merged

MINOR: [R] Fix a package namespace warning#43737
jonkeane merged 3 commits intoapache:mainfrom
jonkeane:fix_dt_warn

Conversation

@jonkeane
Copy link
Copy Markdown
Member

@jonkeane jonkeane commented Aug 17, 2024

Oops, I should have caught this in #43633 Removes data.table:: since the namespace is loaded. Also fix some linting errors and free up space on the force tests run.

@jonkeane jonkeane merged commit fbac12c into apache:main Aug 17, 2024
@jonkeane jonkeane removed the awaiting committer review Awaiting committer review label Aug 17, 2024
@conbench-apache-arrow
Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit fbac12c.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant