Problem/Motivation
- Check and align comments at
Drupal\k8s\Controller\ApiControllerInterface - The following is a list of CamelCase resource names (the other one is NOT CamelCame):
ConfigMapsCronJobDaemonSetLimitRangeReplicaSetServiceAccountStatefulSet
- See also the Glossary page: https://kubernetes.io/docs/reference/glossary/?all=true
- Format:
/** * Update all K8s <Resource Name or ResourceName>. * * @param string $cloud_context * Cloud context string. * * @return \Symfony\Component\HttpFoundation\RedirectResponse * A redirect response object that may be returned by the controller. */ - For example:
/** * Update all K8s LimitRanges. * * @param string $cloud_context * Cloud context string. * * @return \Symfony\Component\HttpFoundation\RedirectResponse * A redirect response object that may be returned by the controller. *//** * Update all K8s Network Policies. * * @param string $cloud_context * Cloud context string. * * @return \Symfony\Component\HttpFoundation\RedirectResponse * A redirect response object that may be returned by the controller. */
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | cloud-2578.diff | 615 bytes | TamakiFujino |
Issue fork cloud-3225218
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:
- 3225218-align-the-comments
changes, plain diff MR !367
Comments
Comment #3
kumikoono commentedComment #4
TamakiFujino commentedPlease review
Comment #6
TamakiFujino commentedComment #7
TamakiFujino commentedComment #8
kumikoono commented@tamaki The patch looks good to me. Thanks!
Comment #9
yas@tamakifujino
Thank you for supplying the patch. I posted a minor comment above. Please check it. Thanks!
Comment #10
yas@tamakifujino
Thank you for the update. I'll merge the patch to
4.xand close this issue as Fixed.Comment #13
yas