Problem/Motivation

From change record https://www.drupal.org/node/3258656 we learn that parts of the feed.bridge.reader.service is deprecated.

#3258435: Remove deprecated feed.reader.* and feed.writer.* services from the container performed the removal for this deprecation in 10.0.x.

However, we need to deprecate the component to warn users of this component (if they exist) that the package won't be further updated.

The way we do that in Composer-land is to mark the package as abandoned.

Proposed resolution

Add abandoned = true to the composer.json file for 9.5.x.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3280399

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Mile23 created an issue. See original summary.

mile23’s picture

Issue summary: View changes
mile23’s picture

Issue summary: View changes
andypost’s picture

Issue tags: +Deprecation Removal
mile23’s picture

mile23’s picture

Title: Remove drupal/core-bridge component per deprecation » Mark drupal/core-bridge 9.5.x as abandoned
Version: 10.0.x-dev » 9.5.x-dev
Component: base system » composer
Status: Closed (duplicate) » Active

Rescoping to marking drupal/core-bridge as abandoned in 9.5.x.

That way we can responsibly warn whoever uses this component that it's about to go away.

mile23’s picture

Issue summary: View changes

hestenet made their first commit to this issue’s fork.

mile23’s picture

Status: Active » Needs review

Set drupal/core-bridge to be abandoned.

Discuss. :-)

andypost’s picture

Issue tags: +Needs change record

Good idea, sadly there's no additional property to point user to the required change record

mile23’s picture

Issue tags: -Needs change record

Added a link to the existing CR in the extra section.

It doesn't help that the CR says "@todo" in the description....

catch’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll

Looks like this needs a rebase.

voleger made their first commit to this issue’s fork.

voleger’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
daffie’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

  • catch committed 9050e30 on 9.5.x
    Issue #3280399 by Mile23, voleger, hestenet, andypost, daffie: Mark...
catch’s picture

Status: Reviewed & tested by the community » Fixed

This is slightly unusual but I think because we've just not done it before. Already removed in 10.0.x

Committed/pushed to 9.5.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

quietone’s picture

asked in #contribute if someone could update the CR, it only has an '@todo' for content. longwave responded, updating the CR and publishing it.