Skip to content

Conversation

@DavisVaughan
Copy link
Member

@DavisVaughan DavisVaughan commented Nov 8, 2025

Deprecationpalooza!

In addition to these, see #7761, #7760, #7758, and #7757, all of which require a bit more individual care.


Revdeps in #7763 show ~150 issues, but 100 of them are due to removal of the already defunct dplyr::id(). People were using mutate(df, col = id) and not listing id as a globalVariable() so now they will get a note about this. I can send all of these people a github issue / email, but I was not planning on doing individual PRs to them. It does seem worth pushing on this though.

Remotes:
    vctrs=url::https://github.com/DavisVaughan/vctrs/archive/test/remote-urls.tar.gz

Plus set to CRAN lifecycle

"97e2bc7a-8a36-4e8f-b963-036934944b6e"

By removing compatibility for this altogether
By removing the warning about it entirely, it was being ignored anyways
@DavisVaughan DavisVaughan force-pushed the feature/advance-deprecations branch from 1807491 to f84976c Compare November 8, 2025 20:02
@DavisVaughan DavisVaughan requested a review from hadley November 11, 2025 17:34
Copy link
Member Author

Choose a reason for hiding this comment

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

It's probably easiest to start with the NEWS to get an overview, then briefly look at any of the standalone commits and see if any jump out at you as warranting extra attention


### Breaking changes

* The following were already deprecated, and are now defunct:
Copy link
Member

Choose a reason for hiding this comment

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

It would be nice to again remind people what the replacements are (I asked claude to do that for me). I also generally include the version number they were deprecated, and the date it was released.

@DavisVaughan DavisVaughan merged commit 64152af into main Nov 12, 2025
12 checks passed
@DavisVaughan DavisVaughan deleted the feature/advance-deprecations branch November 12, 2025 17:14
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.

3 participants