Problem/Motivation

  • Check and align the comments at Drupal\k8s\Entity\K8s<ResourceName>ViewBuilder
  • Related issue: https://www.drupal.org/project/cloud/issues/3225220
  • CamelCase resource names (the other one is NOT CamelCame):
    • ConfigMaps
    • CronJob
    • DaemonSet
    • LimitRange
    • ReplicaSet
    • ServiceAccount
    • StatefulSet
  • See also the Glossary page: https://kubernetes.io/docs/reference/glossary/?all=true
  • Format:
    /**
     * Provides the K8s <Resource Name or Resourcename> view builders.
     */
    
  • For example:
    /**
     * Provides the K8s LimitRange view builders.
     */
    
    /**
     * Provides the K8s Network Policy view builders.
     */
    

Issue fork cloud-3225272

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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yas created an issue. See original summary.

yas’s picture

Issue summary: View changes
yas’s picture

Issue summary: View changes
yas’s picture

Issue summary: View changes
TamakiFujino’s picture

Assigned: Unassigned » TamakiFujino
Status: Active » Needs work

TamakiFujino’s picture

Status: Needs work » Needs review
yas’s picture

Status: Needs review » Reviewed & tested by the community

@tamakifujino

Thank you for the refactoring. It looks good to me. I'll merge the patch to 4.x and close this issue as Fixed.

  • yas committed 995bea3 on 4.x authored by TamakiFujino
    Issue #3225272 by TamakiFujino, yas: Align the comments at Drupal\k8s\...

yas’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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