Ke-vin-S opened a new pull request, #8715: URL: https://github.com/apache/incubator-devlake/pull/8715
### Summary What does this PR do? Fix deployment finish time calculation in the GitHub GraphQL plugin. Previously, DevLake used the latest deployment status timestamp (often inactive) as the deployment completion time. However, GitHub marks a deployment as inactive when it is superseded by a newer successful deployment, which may occur significantly later than the actual success event. This PR introduces a new nullable field finished_date in GithubDeployment and implements success-priority logic: Use the latest success status timestamp if present. - Otherwise, fall back to the latest terminal status timestamp. - Domain conversion now uses finished_date to populate TaskDatesInfo.FinishedDate and calculate duration. - This ensures deployment metrics (e.g., PR Deploy Time) reflect the actual completion time. ### Does this close any open issues? Closes #8654 ### Other Information - Adds a backward-compatible migration for finished_date. - Preserves existing fields (LatestStatusState, LatestUpdatedDate) without changing semantics. - No breaking changes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
