-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Synthesize a command for a Run #4499
Conversation
…where the command was invoked
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Ignored Deployments
|
🟢 CI successful 🟢Thanks |
func newVercelRunCreatePayload(runsummary *RunSummary) *vercelRunPayload { | ||
startTime := int(runsummary.ExecutionSummary.startedAt.UnixMilli()) | ||
var context = "LOCAL" | ||
func (rsm *Meta) newVercelRunCreatePayload() *vercelRunPayload { |
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.
If it's going into the spaces payload, it should be part of the Run Summary. It seems like good information regardless. If you're good with that, we can merge this in and I'll update in my PR
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.
Added to execution summary
pkgInferencePath
to be aRelativeSystemPath
(note "" token for not set)repositoryPath
in Run Payloadint64
since they millisecondsNote merge base is #4495
Testing Instructions
Added a new test for command syntheses