Skip to content
This repository was archived by the owner on Dec 2, 2025. It is now read-only.

Update transfer handler options#1

Merged
Skarlso merged 1 commit intomainfrom
phoban01/update-options
Nov 14, 2022
Merged

Update transfer handler options#1
Skarlso merged 1 commit intomainfrom
phoban01/update-options

Conversation

@phoban01
Copy link
Copy Markdown
Contributor

Description

This PR adds the correct handler options for the transfer command. It also adds a resolve option for both the target and source repo.

It also fixes a small issue with the semver handling by switch from .String() to .Original(). The semver library will return a strictly correct semver (without the leading v) meaning that the tag could be wrong.

Follow ups:

  • The semver should really be optional, by default we should list the images on both source and dest, then copy anything from the source that is not present on dest.
  • We should support adding resolvers for any repository.
    Signed-off-by: Piaras Hoban phoban01@gmail.com

Signed-off-by: Piaras Hoban <phoban01@gmail.com>
@phoban01 phoban01 requested a review from Skarlso November 14, 2022 10:35
@Skarlso Skarlso merged commit 5faee20 into main Nov 14, 2022
@phoban01 phoban01 deleted the phoban01/update-options branch March 7, 2023 10:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants