Hyphenate 4 words that start with 'un'

Created on 21 November 2023, 7 months ago
Updated 20 June 2024, 8 days ago

Problem/Motivation

🌱 [meta] Fix missing hyphens for prefixes - Words starting with "un" Active contains a list of words that are flagged by cspell that start with un-. Numerous of the words are also not in other standard English dictionaries.

Proposed resolution

Hyphenate these words. They are in comments or a string in a test.

    unsanitized
    unserialization
    unserializes
    unserializing

Remaining tasks

Review

User interface changes

TBD

API changes

TBD

Data model changes

TBD

Release notes snippet

N/A

πŸ“Œ Task
Status

Needs review

Version

11.0 πŸ”₯

Component
OtherΒ  β†’

Last updated about 11 hours ago

Created by

πŸ‡ΊπŸ‡ΈUnited States bhumboldt

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @bhumboldt
  • πŸ‡ΊπŸ‡ΈUnited States xjm
  • πŸ‡ΊπŸ‡ΈUnited States xjm
  • πŸ‡ΊπŸ‡ΈUnited States xjm
  • First commit to issue fork.
  • Merge request !8421Hyphenate 4 words that start with 'un' β†’ (Open) created by quietone
  • Issue was unassigned.
  • Status changed to Needs review 13 days ago
  • πŸ‡³πŸ‡ΏNew Zealand quietone New Zealand

    Created an MR for what should be an easy set of words.

  • πŸ‡¬πŸ‡§United Kingdom joachim

    These seem like weird words to hyphenate.

    unsanitary is a word without a hyphen, so it doesn't make sense for unsanitized to need one.

    And the PHP function is called unserialize(), so again, it's weird to put a hyphen in text.

  • πŸ‡ΊπŸ‡ΈUnited States xjm

    #8 also reminds me that the original intent in the meta was to figure out why certain PHP/git/etc. language terms and their derivatives were missing, and/or if they should be added to the Drupal dictionary.

  • Status changed to Needs work 8 days ago
  • The Needs Review Queue Bot β†’ tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

    This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

    Consult the Drupal Contributor Guide β†’ to find step-by-step guides for working with issues.

  • Status changed to Needs review 8 days ago
  • πŸ‡³πŸ‡ΏNew Zealand quietone New Zealand

    I think instead of figuring out why cspell has a word or not in a particular dictionary we should focus on what works for Drupal. Do we a) change the code/comments or b) add a given word to the drupal dictionary or c) make a PR to have it added to the a given dictionary.

    Of those choices, what should happen to the 4 words in this issue?

Production build 0.69.0 2024