Problem/Motivation

Fix a collection of words that are used in test modules.

Steps to reproduce

Proposed resolution

  1. bodyformat
  2. bodyvalue
  3. cleartype
  4. httpkernel
  5. kakec
  6. labely
  7. madeup
  8. minipager
  9. multipleforms
  10. mooooh
  11. parens
  12. testdescription
  13. testid
  14. testitem
  15. testlist
  16. testvalue
  17. yowser

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3395891

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone CreditAttribution: quietone at PreviousNext commented
Title: Fix speeling for words in test modules » Fix spelling for words in test modules
quietone’s picture

quietone CreditAttribution: quietone at PreviousNext commented
Issue summary: View changes

quietone opened merge request !5090

quietone’s picture

quietone CreditAttribution: quietone at PreviousNext commented
Issue summary: View changes
Status: Active » Needs review
smustgrave’s picture

smustgrave CreditAttribution: smustgrave at Mobomo commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

Reviewed the text changes and reads fine.

Also words in the IS match the MR.

quietone’s picture

quietone CreditAttribution: quietone at PreviousNext commented

Failure in unrelated test, Drupal\FunctionalJavascriptTests\Core\Field\TimestampFormatterWithTimeDiffTest::testTimestampFormatterWithTimeDiff. #3400150: [random test failure] TimestampFormatterWithTimeDiffTest::testTimestampFormatterWithTimeDiff.

Leaving at RTBC

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

All the changes here make sense to me and this is scoped at about the right size for these sorts of changes - not too big to review but not too small that we have lots of similar issues.

Committed and pushed 7e75964e69 to 11.x and 394b8fb2c5 to 10.2.x. Thanks!

  • longwave committed 394b8fb2 on 10.2.x 
    Issue #3395891 by quietone: Fix spelling for words in test modules

(...

  • longwave committed 7e75964e on 11.x 
    Issue #3395891 by quietone: Fix spelling for words in test modules

quietone closed merge request !5090

Status: Fixed » Closed (fixed)

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