Problem/Motivation

In #3250342: Deprecate "experimental: true" in .info.yml we deprecated the use of key-value pair experimental: true in info.yml files.

This issue is here to remove the deprecation and all of its test-files when the drupal 11.0.x branch is opened

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3321634

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:

About issue forks

Comments

Spokje created an issue. See original summary.

spokje’s picture

spokje
Primary language Dutch
commented
Status: Active » Postponed

Postponed on the opening of a 11.0.x-dev branch.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Title: Remove legacy experimental:true and its deprecation and test-files » [11.0] Remove legacy experimental:true and its deprecation and test-files
spokje’s picture

spokje
Primary language Dutch
commented
Version: 10.1.x-dev » 11.0.x-dev
catch’s picture

catch
he/him
Primary language English
commented
Title: [11.0] Remove legacy experimental:true and its deprecation and test-files » [11.x] Remove legacy experimental:true and its deprecation and test-files
Issue tags: +Major version only
catch’s picture

catch
he/him
Primary language English
commented
Version: 11.0.x-dev » 11.x-dev

Spokje opened merge request !6856

spokje’s picture

spokje
Primary language Dutch
commented
Status: Postponed » Needs review
andypost’s picture

andypost
he/him
Primary language Russian
commented

One nit, please restore comment as it was in https://git.drupalcode.org/project/drupal/-/commit/94230c5aa0582de0fe2e9...

spokje’s picture

spokje
Primary language Dutch
commented

@andypost: These long links are really meh, and I think you might have missed adding the specific line number(s) to it, but I _think_ you're referring to this one in : https://git.drupalcode.org/project/drupal/-/commit/94230c5aa0582de0fe2e9... in core/modules/system/tests/modules/theme_page_test/theme_page_test.module?

andypost’s picture

andypost
he/him
Primary language Russian
commented

Yes, added suggestion to apply

quietone’s picture

quietone commented
Parent issue: » #3295574: [meta] Remove deprecated classes, methods, procedural functions and code paths outside of deprecated modules on the Drupal 11 branch
quietone’s picture

quietone commented
Title: [11.x] Remove legacy experimental:true and its deprecation and test-files » Remove legacy experimental:true and its deprecation and test-files
spokje’s picture

spokje
Primary language Dutch
commented

Yes, added suggestion to apply

Thanks @andypost, suggestion applied.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Ready to go

  • catch committed aa649dc9 on 10.3.x 
    Issue #3321634 by Spokje, andypost: Remove legacy experimental:true and...

  • catch committed 5c254c67 on 11.x 
    Issue #3321634 by Spokje, andypost: Remove legacy experimental:true and...
catch’s picture

catch
he/him
Primary language English
commented
Version: 11.x-dev » 10.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!

Spokje closed merge request !6856

Status: Fixed » Closed (fixed)

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