Problem/Motivation

Follow-up #3155400: Change $requirements["$file file writeable"] to $requirements["$file file writable"].

As title.

Proposed resolution

Add writeable to the list of flagged words.
Add a "//cspell:ignore" line near the top of the file for 'writeable' and the two words ignored by "// cSpell:disable-next-line" lines. Then remove the "// cSpell:disable-next-line" lines.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3155411

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

jungle created an issue. See original summary.

shaktik’s picture

shaktik
he/him
Primary language Hindi
Location New Delhi
commented

Hi @jungle,

git clone --branch 10.0.x https://git.drupalcode.org/project/drupal.git

.git
index.php

just clone and get only index.php file.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Postponed

10.0.x is not open for development yet. This issue cannot be completed until the branch is open and #3138595: Deprecate PhpStorage ::writeable() methods for removal in Drupal 11 with no replacement is done, so marking this as postponed.

Version: 10.0.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

quietone’s picture

quietone commented
Status: Postponed » Active
Parent issue: » #3122088: [Meta] Remove spelling errors from dictionary.txt and fix them

quietone opened merge request !8469

quietone’s picture

quietone commented
Component: base system » other
Issue summary: View changes
Status: Active » Needs review
Issue tags: +cspell error

The method \Drupal\Tests\StreamCapturer::filter has two "// cSpell:disable-next-line" lines. While I usually follow the existing pattern in a file adding another // cSpell:disable-next-line" in a method with 6 lines of code just add clutter. Therefor, I moved all the words to a single ignore line.

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

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Tested my theory and I was wrong so restored to what it was. LGTM

quietone’s picture

quietone commented

  • catch committed c3e68680 on 11.x 
    Issue #3155411 by quietone, smustgrave, jungle, longwave: Add "writeable...

  • catch committed f271302c on 11.0.x 
    Issue #3155411 by quietone, smustgrave, jungle, longwave: Add "writeable...
catch’s picture

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

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

catch closed merge request !8469

Status: Fixed » Closed (fixed)

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