Skip to content

updating artifact not found error message#1628

Merged
eggyhead merged 3 commits intomainfrom
eggyhead/update-getartifact-errmessage
Jan 18, 2024
Merged

updating artifact not found error message#1628
eggyhead merged 3 commits intomainfrom
eggyhead/update-getartifact-errmessage

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
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.

4 participants