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

Add support for updatedRefs in WebhookPayload #2169

Merged
merged 2 commits into from
Mar 29, 2024

Conversation

Wauplin
Copy link
Contributor

@Wauplin Wauplin commented Mar 29, 2024

Related to huggingface/hub-docs#1252 cc @coyotte508

Wehbook payload now sends a list of the updated refs. This is useful to react on new commits on specific pull requests, new tags and new branches. See https://huggingface.co/docs/hub/webhooks#code-changes for more details.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.88%. Comparing base (9ced134) to head (60a3d4f).

❗ Current head 60a3d4f differs from pull request most recent head d60936d. Consider uploading reports for the commit d60936d to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2169      +/-   ##
==========================================
- Coverage   82.90%   82.88%   -0.03%     
==========================================
  Files         103      103              
  Lines        9698     9703       +5     
==========================================
+ Hits         8040     8042       +2     
- Misses       1658     1661       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Member

@julien-c julien-c left a comment

Choose a reason for hiding this comment

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

nice!

@julien-c
Copy link
Member

let's try to grow the usage of webhook btw, i feel like this is powerful but under-exploited

@Wauplin Wauplin merged commit 20884c3 into main Mar 29, 2024
16 checks passed
@Wauplin Wauplin deleted the accept-update-refs-in-webhook-payload branch March 29, 2024 14:45
@Wauplin
Copy link
Contributor Author

Wauplin commented Mar 29, 2024

let's try to grow the usage of webhook btw

Maybe worth implementing #1808?

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