Problem/Motivation

@larowlan did a great review on #3439923: Add recipes api as experimental API to core

Proposed resolution

Do all the easy things in one issue.

List of things addressed here:

  • Drop PHP 8.1 requirement - it is redundant
  • References to annotations instead of attributes
  • Improved docs for the config_name argument in \Drupal\Core\Config\Action\ConfigActionManager::applyAction
  • Early return in \Drupal\Core\Config\Checkpoint\CheckpointStorage::checkpoint()
  • Use Sequentially to avoid odd code in validateExtensionIsAvailable

Remaining tasks

User interface changes

API changes

Data model changes

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

alexpott created an issue. See original summary.

alexpott credited larowlan.

alexpott’s picture

alexpott’s picture

Issue summary: View changes
alexpott’s picture

Status: Active » Needs review

alexpott’s picture

Version: 11.x-dev » 10.3.x-dev
Status: Needs review » Fixed

Committed and pushed 0f5f94f92a3 to 11.x and 04e1bb20499 to 10.3.x. Thanks!

  • alexpott committed 04e1bb20 on 10.3.x
    Issue #3442002 by alexpott, larowlan: @larowlan review on #3439923
    
    (...

  • alexpott committed 0f5f94f9 on 11.x
    Issue #3442002 by alexpott, larowlan: @larowlan review on #3439923
    
alexpott’s picture

  • 45c64852 committed on patch
    Update recipe 10.3.x patch 04e1bb20 Issue #3442002 by alexpott, larowlan...

  • f1f3bc12 committed on patch
    Update recipe 11.x patch 0f5f94f9 Issue #3442002 by alexpott, larowlan...

Status: Fixed » Closed (fixed)

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