Skip to content

update require-version-in-deprecation for new deprecation policy#102

Merged
paulius-valiunas merged 7 commits intomainfrom
deprecation-version-bump
May 28, 2025
Merged

update require-version-in-deprecation for new deprecation policy#102
paulius-valiunas merged 7 commits intomainfrom
deprecation-version-bump

Conversation

@paulius-valiunas
Copy link
Copy Markdown
Collaborator

@paulius-valiunas paulius-valiunas commented May 14, 2025

This rule has multiple uses:

  1. Just like before, ensure that developers add a reason to their deprecations.
  2. If the a string value is passed into the addVersion option, the rule will add that version to every deprecation comment that doesn't already contain a version in the expected place.
  3. If the removeOldDates option is enabled, the rule will check the dates in deprecation comments and replace any expired ones with "might be removed in next major version".

The first use is intended for developer machines and PR pipelines. The rest are intended for automating the release process.

@paulius-valiunas paulius-valiunas marked this pull request as ready for review May 16, 2025 11:14
ben-polinsky
ben-polinsky previously approved these changes May 16, 2025
Copy link
Copy Markdown
Collaborator

@ben-polinsky ben-polinsky left a comment

Choose a reason for hiding this comment

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

Regex is a bit gnarly but the comprehensive tests ease my fears.

@paulius-valiunas paulius-valiunas dismissed ben-polinsky’s stale review May 20, 2025 11:32

significant new changes pushed

@paulius-valiunas paulius-valiunas merged commit 7afeae2 into main May 28, 2025
4 checks passed
@paulius-valiunas paulius-valiunas deleted the deprecation-version-bump branch May 28, 2025 10:52
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