Skip to content

Implement specify_df()#37

Merged
jonthegeek merged 2 commits into
mainfrom
fix-4-specify_df
Oct 8, 2025
Merged

Implement specify_df()#37
jonthegeek merged 2 commits into
mainfrom
fix-4-specify_df

Conversation

@jonthegeek
Copy link
Copy Markdown
Collaborator

Closes #4

@jonthegeek jonthegeek requested a review from Copilot October 8, 2025 10:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a new specify_df() function that allows users to specify the desired format for data frames by combining column specifications. The implementation includes comprehensive error handling and validation utilities.

Key changes:

  • Added specify_df() function for creating data frame specifications
  • Implemented validation utilities for argument checking (.check_args_named(), .check_args_spec())
  • Added error handling infrastructure with standardized error classes
  • Updated documentation with cross-references between specification functions

Reviewed Changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated no comments.

Show a summary per file
File Description
R/specify_df.R Core implementation of the new specify_df() function
R/aaa-conditions.R Error handling utilities and validation functions
tests/testthat/test-specify_df.R Comprehensive test suite for specify_df() functionality
tests/testthat/test-aaa-conditions.R Tests for error handling and validation utilities
R/specify_fct.R Updated documentation to reference the new function
man/*.Rd Generated documentation files for the new functions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

hrmn_fct_spec --> hrmn_spec_fct

hrmn_df_spec --> hrmn_spec_df
@jonthegeek jonthegeek enabled auto-merge (squash) October 8, 2025 10:31
@jonthegeek jonthegeek merged commit e46a514 into main Oct 8, 2025
7 checks passed
@jonthegeek jonthegeek deleted the fix-4-specify_df branch October 8, 2025 10:35
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.

Implement specify_df()

2 participants