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

updating artifact not found error message #1628

Merged
merged 3 commits into from
Jan 18, 2024

Conversation

eggyhead
Copy link
Contributor

What are we doing?

We want to be more descriptive in the reason why users might experience an artifact not being found. This error message update helps direct users to troubleshoot using the FAQ docs and common reasons for this issue.

How are we doing it?

Updating the ArtifactNotFoundError in getArtifactPublic and getArtifactInternal to include more information.

How do I test?

Use the updated toolkit in a workflow using an outdated artifact or an artifact uploaded using a previous version. Expect the new error message.

@eggyhead eggyhead requested a review from a team as a code owner January 18, 2024 04:14
@eggyhead eggyhead merged commit 5212cb5 into main Jan 18, 2024
14 checks passed
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.

None yet

4 participants