Skip to content

remove deprecated print method, and adjust NAMESPACE. Add startup mes… #102

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 10, 2025

Conversation

stemangiola
Copy link
Collaborator

@stemangiola stemangiola commented Jul 10, 2025

.

@stemangiola stemangiola requested a review from Copilot July 10, 2025 03:48
Copy link

@Copilot 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 removes deprecated print methods and associated documentation, updates the package namespace, and adds a startup message informing users of external printing handling.

  • Remove old Rd documentation and R code for print.SummarizedExperiment and tbl_format_header.
  • Clean up NAMESPACE by dropping obsolete S3 registrations and imports.
  • Add a startup message in .onAttach() directing users to the new tidyprint workflow.

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
man/tbl_format_header.Rd Deleted deprecated header format documentation
man/formatting.Rd Deleted deprecated print documentation
R/zzz.R Added packageStartupMessage to alert users about external printing
R/utilities.R Removed unused internal add_attr() helper
R/print_method.R Removed deprecated printing methods
R/pillar_utilities.R Removed internal comment-wrapping utilities
R/methods.R Removed custom show method override
NAMESPACE Dropped S3 methods and pillar imports related to removed code
DESCRIPTION Bumped version from 1.15.1 to 1.15.2
Comments suppressed due to low confidence (4)

DESCRIPTION:4

  • [nitpick] After bumping the version, please add a corresponding entry to NEWS.md or your changelog detailing the deprecated print removal and the new startup message.
Version: 1.15.2

R/utilities.R:1289

  • Ensure that add_attr() is not referenced elsewhere after its removal; if it's still used by other utilities or tests, this could break functionality.
is_filer_columns_in_column_selection <- function(.data, ...) {

…documentation on S3 method structure, enhanced ggplot2 compatibility, and user guidance towards tidyprint for visualization.
@stemangiola stemangiola merged commit 275b3f3 into master Jul 10, 2025
3 checks passed
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.

1 participant