ROSAENG-00000 | docs: expand CLI abbreviations for product documentation compliance (OSDOCS-16406) - #3345
Conversation
openshift-ci-robot
added
the
jira/valid-reference
|
@EricPonvelle: This pull request references OSDOCS-16406 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThis pull request updates Cobra command help text across create, delete, unlink, list, link, and verify commands. Several create commands now spell out DNS, OIDC, IAM, and identity provider terms, and 🚥 Pre-merge checks | ✅ 13 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (13 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@cmd/create/idp/cmd.go`:
- Around line 92-93: Tighten the help text in the command definition by updating
the `Short` and `Long` descriptions to use clearer wording: change the phrase in
the `cmd/create/idp/cmd.go` command metadata so it reads naturally as “Add
identity provider (IDP) for a cluster” and rephrase the longer description to
avoid “identity providers” in the awkward plural form. Keep the fix confined to
the `Short` and `Long` fields in the `createIDPCmd` command setup.
In `@cmd/create/oidcconfig/cmd.go`:
- Around line 57-58: Update the user-facing long description in the OIDC config
command metadata so the help text uses “an S3 bucket” instead of “a S3 bucket.”
Make this change in the command definition where the OpenID Connect config
description is built, keeping the rest of the wording unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Central YAML (inherited)
Review profile: CHILL
Plan: Enterprise
Run ID: 92ddc5da-4fd9-4965-a78c-1522ad0a5e4d
📒 Files selected for processing (5)
cmd/create/dnsdomains/cmd.gocmd/create/idp/cmd.gocmd/create/ocmrole/cmd.gocmd/create/oidcconfig/cmd.gocmd/create/oidcprovider/cmd.go
db3f59a to
0efd712
Compare
bhardesty
left a comment
bhardesty
left a comment
There was a problem hiding this comment.
Just noticed one small spacing issue.
ab7f08f to
68ad034
Compare
|
@EricPonvelle The commit message format needs to match the repo convention: |
EricPonvelle
changed the title
openshift-ci-robot
removed
the
jira/valid-reference
|
@EricPonvelle: No Jira issue is referenced in the title of this pull request. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
68ad034 to
d3d708f
Compare
|
@olucasfreitas Thanks for that review! I think I addressed everything correctly that you pointed out, and I updated the commit message. Let me know if you spot anything else I need to fix. |
EricPonvelle
changed the title
f104178 to
78b7e2f
Compare
…ion compliance (OSDOCS-16406)
78b7e2f to
ca87448
Compare
|
@EricPonvelle: EricPonvelle unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers openshift-sustaining-engineers. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/override ci/prow/lint |
|
@EricPonvelle: EricPonvelle unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight openshift-staff-engineers openshift-sustaining-engineers. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/override ci/prow/lint |
|
@olucasfreitas: Overrode contexts on behalf of olucasfreitas: ci/prow/govulncheck, ci/prow/lint, ci/prow/security DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@EricPonvelle Just needs a |
|
/override ci/prow/lint |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: EricPonvelle, olucasfreitas The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
Bot
added
the
approved
|
@olucasfreitas: Overrode contexts on behalf of olucasfreitas: ci/prow/govulncheck, ci/prow/lint, ci/prow/security DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@EricPonvelle: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
4 participants
This PR addresses:
Detailed Description of the Issue
This PR addresses some language issues that were not in compliance with product documentation standards. Since we generate our CLI documentation from this repo, the updates had to be done here. To that end, I have changed the descriptions on a few ROSA commands so that they have the full names rather than acronyms.
Type of Change
Developer Verification Checklist
[JIRA-TICKET] | [TYPE]: <MESSAGE>.make install-hookshas been run in this clone.make testpasses.make lintpasses.make rosapasses.Summary by CodeRabbit
create,delete,list,link,unlink, andverify) with clearer, standardized capitalization and phrasing.