-
Notifications
You must be signed in to change notification settings - Fork 4
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: send RunPart
metadata to upload endpoint
#57
base: main
Are you sure you want to change the base?
feat: send RunPart
metadata to upload endpoint
#57
Conversation
fargito
commented
Jan 15, 2025
- chore(rust): bump toolchain from 1.79.0 to 1.84.0
- chore: add missing spellings
- feat: add platform_slug to upload metadata
- feat: add platform_run_part to providers
06848b2
to
9b963a3
Compare
fb53a4f
to
dfdd096
Compare
25cd578
to
f87e92e
Compare
@adriencaccia ready for a first review |
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.
A few minor changes 💪
Make sure to fixup without force-pushing so we can compare the diff easily
f87e92e
to
d89c070
Compare
7490e74
to
702328e
Compare
@adriencaccia @Arthur I updated and also added some tests |
702328e
to
ed81950
Compare
@art049 sorry about the force-pushes. Here is the comparison link between the reviewed and the latest version: https://github.com/CodSpeedHQ/runner/compare/f87e92ed46b24c5635282521365023e1a31571d5..ed81950a62afe82f0c8d2839fa940d46ea9bc5a3 |
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.
As discussed together, it would be best to just keep job-index and job-total in the last part of the matrix part id
@adriencaccia I removed some values from the metadata, can you release a new beta version? |