Skip to content
Snippets Groups Projects

Issue #3444468: Updates documentation and test scripts for clarity

Issue #3444468: Updates documentation and test scripts for clarity
All threads resolved!

I wanted to make the docs a little easier to read and do a little test script tidying.

  1. The main README.md has more styling a few language changes
  2. The script README.md has line breaks removed
  3. The 10000-item script has been renamed for clarity. Maybe it's my bad eyes, but TestDo10kThings.php and TestDo10Things.php were harder to distinguish.
  4. The language of the script logs and comments has been more aligned with their intention or content.
Edited by Christian Burk

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Steve Wirt
  • Steve Wirt
  • Steve Wirt
  • A couple of needed changes, and the rest are all good.

    I am on the fence about the line lenght. MD doesn't exactly play nicely with drupal.org's code style.

  • added 1 commit

    • 41ccca4f - Updates code for PHPCS errors

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • ec4b1609 - Adds the new file name to prependPath()

    Compare with previous version

  • Christian Burk requested review from @swirtMiles

    requested review from @swirtMiles

  • Steve Wirt
  • Steve Wirt
  • Steve Wirt changed title from Updates documentation and test scripts for clarity to Issue #3444468: Updates documentation and test scripts for clarity

    changed title from Updates documentation and test scripts for clarity to Issue #3444468: Updates documentation and test scripts for clarity

  • Steve Wirt resolved all threads

    resolved all threads

  • Steve Wirt approved this merge request

    approved this merge request

  • Approved and ready for merge if TestDo10000Things can be run

  • added 1 commit

    • c5579a7f - Adds spelling configuration for words we want to use.

    Compare with previous version

  • Christian Burk reset approvals from @swirtMiles by pushing to the branch

    reset approvals from @swirtMiles by pushing to the branch

  • added 1 commit

    • 496837c0 - Changes spelling to pass cspell without warnings

    Compare with previous version

  • Passed with no warnings!

    Screenshot_2024-04-30_at_2.43.08_PM

  • Newly named script ran successfully: Screenshot_2024-04-30_at_2.44.38_PM

  • Christian Burk started a merge train

    started a merge train

  • Please register or sign in to reply
    Loading