Skip to content

Redefine default test location for leapp-repository#52

Open
danmyway wants to merge 1 commit intodevelfrom
resolve-default-repo-for-leapp-tests
Open

Redefine default test location for leapp-repository#52
danmyway wants to merge 1 commit intodevelfrom
resolve-default-repo-for-leapp-tests

Conversation

@danmyway
Copy link
Owner

@danmyway danmyway commented Sep 14, 2023

  • without -g/--git the script was resolving the default test repository url poorly, user forced to specify as commandline option
  • additionally user was unable to select a branch/git reference when specifying the leapp-tests location on the commandline
  • with the fix the default leapp-tests repository is defined without the git argument needed
  • make git option accept four args - baseurl, repo owner, repo name and git branch/reference to checkout
  • add simple test covering the git argument
  • resolves Handle the default leapp-test repository better #37
  • additional changes made by pre-commit hook

* without `-g/--git` the script was resolving the default test
  repository url poorly, user forced to specify as commandline option
* additionally user was unable to select a branch/git reference when specifying the
  leapp-tests location on the commandline
* with the fix the default leapp-tests repository is defined without the
  git argument needed
* make git option accept four args - baseurl, repo owner, repo name and
  git branch/reference to checkout
* add simple test covering the git argument
* resolves 37

* additional changes made by pre-commit hook

Signed-off-by: Daniel Diblik <ddiblik@redhat.com>
@danmyway danmyway force-pushed the resolve-default-repo-for-leapp-tests branch from f6bb89f to 94d2567 Compare March 21, 2024 09:17
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.

Handle the default leapp-test repository better

1 participant