Skip to content
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

Update deprecate-a-chocolatey-package.md #949

Closed
wants to merge 4 commits into from

Conversation

chtof
Copy link

@chtof chtof commented Feb 10, 2024

Description Of Changes

Added instructions where a deprecated package is not being superseded and so, it's not possible to provide a dependency to a new package.

Motivation and Context

This procedure does not work as-is when a package is not being superseded.

Testing

Tested on the deprecation of https://community.chocolatey.org/tjs, https://community.chocolatey.org/tjs.install, https://community.chocolatey.org/tjs.portable packages.

Change Types Made

  • Minor documentation fix (typos etc.).
  • Major documentation change (refactoring, reformatting or adding documentation to existing page).
  • New documentation page added.
  • The change I have made should have a video added, and I have raised an issue for this.
    • Issue #

Change Checklist

  • Requires a change to menu structure (top or left-hand side)/
  • Menu structure has been updated

Related Issue

(#948)

Update to take into account packages which are not being superseeded and so, when it's not possible to provide a dependency and needs to provide an empty package.
@vexx32 vexx32 requested review from gep13 and pauby March 21, 2024 18:19
@vexx32
Copy link
Member

vexx32 commented Mar 21, 2024

Per the comments in the issue #948 we don't believe this is an appropriate use of a deprecation, and the expectation is that all package versions simply become unlisted if the software is no longer available.

Deprecations are intended to always have a superseding package to point to.

@vexx32 vexx32 closed this Mar 21, 2024
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