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

PTFE-1575 Upgrade githubkit to latest version #555

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

tcarmet
Copy link
Contributor

@tcarmet tcarmet commented Mar 18, 2024

GitHub most likely made some updates and small tweaks to the data it sends with webhook, and the runner-manager is currently raising some 422 errors when receiving some of them.

Updating githubkit to the latest version available to benefit from the latest changes/updates within GitHub's API.

@tcarmet tcarmet force-pushed the bugfix/upgrade-githubkit branch 5 times, most recently from 72360ba to f3302e9 Compare March 19, 2024 20:45
@codecov-commenter
Copy link

codecov-commenter commented Mar 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.95%. Comparing base (13d7daf) to head (e6f0ad2).

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #555       +/-   ##
===========================================
+ Coverage   67.71%   85.95%   +18.23%     
===========================================
  Files          31       31               
  Lines        1211     1210        -1     
===========================================
+ Hits          820     1040      +220     
+ Misses        391      170      -221     
Flag Coverage Δ
api 67.52% <93.54%> (-0.20%) ⬇️
unit 70.82% <83.87%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@tcarmet tcarmet force-pushed the bugfix/upgrade-githubkit branch 2 times, most recently from e6f0ad2 to 0e20268 Compare March 19, 2024 20:56
@tcarmet tcarmet marked this pull request as ready for review March 19, 2024 20:59
@tcarmet tcarmet requested a review from a team as a code owner March 19, 2024 20:59
@tcarmet tcarmet changed the title Upgrade githubkit to latest version PTFE-1575 Upgrade githubkit to latest version Mar 19, 2024
@tcarmet tcarmet merged commit 27e1778 into main Mar 20, 2024
12 checks passed
@tcarmet tcarmet deleted the bugfix/upgrade-githubkit branch March 20, 2024 16:21
tcarmet added a commit that referenced this pull request Apr 2, 2024
…tamp (#564)

Following #555, an assertion error was occuring due to the assumption
that both the `created_at` and `started_at` fields in the `workflow_job`
webhooks were `datetime` instead of `str`. This, following the pattern
in #555, converts the `str` value of the webhook into `datetime` before
the assertion.
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.

3 participants