Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(.github): add --force option to vcs import #5478

Merged
merged 1 commit into from
Nov 25, 2024
Merged

Conversation

youtalk
Copy link
Member

@youtalk youtalk commented Nov 25, 2024

Description

This PR adds the –force option because there are cases where the import fails when executing the vcs import of autoware-nightly.repos.

$ vcs import --help
usage: vcs import [-h] [--input FILE_OR_URL] [--force] [--shallow] [--recursive] [--retry N] [--skip-existing] [--debug] [-w N] [--repos] [path]

Import the list of repositories

optional arguments:
-h, --help show this help message and exit

"import" command parameters:
--input FILE_OR_URL Where to read YAML from (default: -)
--force Delete existing directories if they don't contain the repository being imported (default: False)
--shallow Create a shallow clone without a history (default: False)
--recursive Recurse into submodules (default: False)
--retry N Retry commands requiring network access N times on failure (default: 2)
--skip-existing Don't overwrite existing directories or change custom checkouts in repos using the same URL (but fetch repos with same URL) (default: False)

Common parameters:
--debug Show debug messages (default: False)
-w N, --workers N Number of parallel worker threads (default: 11)
--repos List repositories which the command operates on (default: False)
path Base path to clone repositories to (default: .)

Tests performed

Not applicable.

Effects on system behavior

Not applicable.

Interface changes

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: Yutaka Kondo <[email protected]>
@youtalk youtalk self-assigned this Nov 25, 2024
@youtalk youtalk added type:github-actions Github actions related. (auto-assigned) type:containers Docker containers, containerization of components, or container orchestration. component:openadkit Issues or Features related to Open AD Kit tag:run-health-check Run health-check labels Nov 25, 2024
@youtalk youtalk marked this pull request as ready for review November 25, 2024 03:08
@youtalk youtalk requested review from xmfcx and mitsudome-r November 25, 2024 03:08
@youtalk youtalk merged commit 0fd30c5 into main Nov 25, 2024
47 checks passed
@youtalk youtalk deleted the add-force-option branch November 25, 2024 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:openadkit Issues or Features related to Open AD Kit tag:run-health-check Run health-check type:containers Docker containers, containerization of components, or container orchestration. type:github-actions Github actions related. (auto-assigned)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants