Skip to content

Conversation

Copy link

Copilot AI commented Jan 14, 2026

Analysis of dplyr dependency issue

Understanding the Problem

The issue "where is the dependency on dplyr?" suggests there may be confusion or problems with how dplyr is declared/imported.

Current State

  • dplyr is listed in DESCRIPTION under Imports (line 34)
  • NAMESPACE has both import(dplyr) (line 35) and specific importFrom(dplyr, ...) (lines 37-41)
  • Code uses dplyr:: prefix for all dplyr function calls
  • cheers_checker.R has @import dplyr roxygen tag
  • Other files use @importFrom dplyr ... roxygen tags

Issues Identified

  • Redundant imports: both @import dplyr and @importFrom dplyr ... exist
  • The @import dplyr in cheers_checker.R causes the entire dplyr package to be imported
  • More specific @importFrom statements exist in other files which is better practice

Proposed Solution

  • Remove @import dplyr from cheers_checker.R
  • Replace with specific @importFrom statements for functions used
  • Ensure NAMESPACE is regenerated properly
  • Verify the package builds and tests pass

Verification

  • Run roxygen2 to regenerate NAMESPACE
  • Check that NAMESPACE only has importFrom statements
  • Build and check the package
Original prompt

whwere is the dependency on dplyr?


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants