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(cli): changes file paths to relative paths for the registry client #19

Merged
merged 2 commits into from
Jun 2, 2022

Conversation

jpower432
Copy link
Contributor

Since the build path is in a disposable directory, it cannot be included
in the "title" annotation on the artifacts. To fix this, relative paths within
the workspace are passed to the oras client and the directory is changed before
processing.

Closes #10

Signed-off-by: Jennifer Power barnabei.jennifer@gmail.com

@jpower432 jpower432 marked this pull request as draft May 26, 2022 15:15
@jpower432
Copy link
Contributor Author

@afflom linting errors fixed. Since I handled the error in the defer statement by logging it, I decided to just put in a the logging solution on a new commit.

The only linting error left will be fixed on #15.

@jpower432 jpower432 marked this pull request as ready for review May 27, 2022 20:23
@sabre1041
Copy link
Contributor

@jpower432 I actually was going to submit an issue related to the annotation title so im glad that it s being tackled as part of this issue

Copy link
Contributor

@sabre1041 sabre1041 left a comment

Choose a reason for hiding this comment

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

This looks good. Let's look to merge #15 first and rebase this PR so that we can confirm the linting errors have been resolved

Since the build path is in a disposable directory, it cannot be included
in the "title" annotation on the artifacts. To fix this, relative paths within
the workspace are passed to the oras client and the directory is changed before
processing.

Closes ortelius#10

Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
@jpower432 jpower432 requested a review from sabre1041 June 1, 2022 15:35
@afflom afflom merged commit 92763ee into ortelius:main Jun 2, 2022
@jpower432 jpower432 deleted the bug/relative-path-titles branch July 30, 2022 00:47
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.

[BUG] absolute paths stored as artifact annotation create unexpected pulling behavior
3 participants