-
Notifications
You must be signed in to change notification settings - Fork 2.2k
feat: add GetArtifactMetadata endpoint to ActionsService #3941
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
feat: add GetArtifactMetadata endpoint to ActionsService #3941
Conversation
|
Please do not update the yaml file in your PRs. If it needs updating, please ping me and I'll update the yaml in its own PR. Thank you. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3941 +/- ##
=======================================
Coverage 92.45% 92.46%
=======================================
Files 203 203
Lines 14980 14990 +10
=======================================
+ Hits 13850 13860 +10
Misses 927 927
Partials 203 203 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
github/actions_artifacts.go
Outdated
|
|
||
| // GetArtifactMetadata gets the metadata for a specific artifact. | ||
| // | ||
| // GitHub API docs: https://docs.github.com/rest/actions/artifacts#get-artifact-metadata |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link seems broken.
|
I guess there is some confusion here.
However, the endpoint added in this PR |
Sorry sir , I will keep in mind from next time. |
Sorry @Not-Dhananjay-Mishra Sir , in the issue section nothing was clear like this. Now i get a clear idea now and i will read the whole description from https://docs.github.com/rest/orgs/artifact-metadata and after i will Raise a PR to this branch only . |
2dcbb7c to
f43d2cf
Compare
Description
This PR adds support for the new Artifact Metadata endpoints as requested in #3917.
Changes
GetArtifactMetadatatogithub/actions_artifacts.go.TestActionsService_GetArtifactMetaDataandTestActionsService_GetArtifactMetaData_notFoundto verify the new endpoint and error handling.openapi_operations.yamlto include the new endpoint.Checklist
test.sh).lint.sh).go generate ./....Fixes #3917