Skip to content

These at-since were improperly added - #4706

Merged
res0nance merged 1 commit into
masterfrom
remove-usageStatistics-at-since
May 4, 2020
Merged

These at-since were improperly added#4706
res0nance merged 1 commit into
masterfrom
remove-usageStatistics-at-since

Conversation

@daniel-beck

@daniel-beck daniel-beck commented May 3, 2020

Copy link
Copy Markdown
Member

Our tooling assumes that nobody would backfill @since TODO for old code, since it's already known what the first release was.

So #4549 improperly added @since TODO to methods that have existed for a decade, and nobody noticed in #4587.

Origin branch (not on dev machine right now), please delete on merge.

@res0nance res0nance added the skip-changelog Should not be shown in the changelog label May 4, 2020
@res0nance res0nance added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label May 4, 2020
@res0nance

Copy link
Copy Markdown
Contributor

Merging once CI passes since this is only documentation amends #4705

@res0nance
res0nance merged commit 6baeb65 into master May 4, 2020
@res0nance
res0nance deleted the remove-usageStatistics-at-since branch May 4, 2020 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback skip-changelog Should not be shown in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants