Skip to content

Cli autocomplete readme#67

Merged
Joeavaikath merged 3 commits into
migtools:oadp-devfrom
Joeavaikath:cli-autocomplete-readme
Sep 8, 2025
Merged

Cli autocomplete readme#67
Joeavaikath merged 3 commits into
migtools:oadp-devfrom
Joeavaikath:cli-autocomplete-readme

Conversation

@Joeavaikath
Copy link
Copy Markdown
Contributor

@Joeavaikath Joeavaikath commented Sep 3, 2025

Description

Add a README around enabling completion support for OADP cli.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update
  • Code refactoring (no functional changes)

Changes Made

  • Adds a new README called COMPLETION.md detailing how users of OADP cli can set up command line tab completions.
  • Adds a PR description template for future PRs.

Related Issues

Additional Notes

Comment thread .github/pull_request_template.md Outdated
@@ -0,0 +1,24 @@
## Description
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++ on the template, IMO we should keep the template across all our OADP repos same and uniform. We have this template in OADP Operator repo https://github.com/openshift/oadp-operator/blob/oadp-dev/.github/pull_request_template.md
We should combine the one proposed in this PR with the one in OADP Operator and use that everywhere

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll bring in the same one to start with and propose the change later then

@Joeavaikath Joeavaikath merged commit 04bb3d4 into migtools:oadp-dev Sep 8, 2025
10 checks passed
@Joeavaikath Joeavaikath deleted the cli-autocomplete-readme branch February 19, 2026 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants