Skip to content

Update PR data Daily collection (2025-10-04T01:13:24Z)#181

Merged
gounthar merged 1 commit into
mainfrom
auto-update-pr-data
Oct 4, 2025
Merged

Update PR data Daily collection (2025-10-04T01:13:24Z)#181
gounthar merged 1 commit into
mainfrom
auto-update-pr-data

Conversation

@gounthar

@gounthar gounthar commented Sep 30, 2025

Copy link
Copy Markdown
Owner

This PR contains automatically updated PR statistics data. Generated by GitHub Actions.

Summary by CodeRabbit

  • Bug Fixes

    • Corrected outdated CI statuses to reflect latest build results, reducing false failure indicators.
    • Updated report links to point to current build logs and artifacts.
  • Chores

    • Refreshed timestamps and status rollups across multiple pull requests.
    • Synchronized status summaries after rebuilds/replays, improving accuracy of displayed outcomes.
    • Maintained visibility for checks still in error where issues persist.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

Updated consolidated JSON metadata entries: multiple statusCheckRollup check-run records had startedAt/completedAt, detailsUrl/targetUrl, and some conclusion fields revised; no code, structural, or exported-entity changes.

Changes

Cohort / File(s) Summary of Changes
Consolidated PR metadata
data/consolidated/all_prs.json, data/consolidated/open_prs.json
Multiple statusCheckRollup entries updated: startedAt and completedAt timestamps changed, detailsUrl/targetUrl values replaced with new Jenkins/AppScan/report URLs, and several conclusion values toggled between failure/error and success. No changes to structure, exported entities, or application logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

I nibble at timestamps, hop through URLs bright,
Updating reports in the hush of the night.
Jenkins trails gleam, conclusions realigned,
A rabbit’s small paw nudged metadata kind. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly indicates the automated daily update of PR data with the relevant timestamp and directly reflects the content of the changeset, making it concise and unambiguous.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch auto-update-pr-data

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gounthar gounthar force-pushed the auto-update-pr-data branch from d1a5a48 to 9a781c2 Compare October 1, 2025 01:27
@gounthar gounthar changed the title Update PR data Daily collection (2025-09-30T01:17:28Z) Update PR data Daily collection (2025-10-01T01:27:31Z) Oct 1, 2025
@gounthar gounthar changed the title Update PR data Daily collection (2025-10-01T01:27:31Z) Update PR data Monthly collection (2025-10-02T01:12:01Z) Oct 2, 2025
@gounthar gounthar force-pushed the auto-update-pr-data branch from 9a781c2 to b4d28a3 Compare October 2, 2025 01:12
@gounthar gounthar changed the title Update PR data Monthly collection (2025-10-02T01:12:01Z) Update PR data Daily collection (2025-10-02T01:14:56Z) Oct 2, 2025
@gounthar gounthar force-pushed the auto-update-pr-data branch from b4d28a3 to a447a7e Compare October 2, 2025 01:15
@gounthar gounthar changed the title Update PR data Daily collection (2025-10-02T01:14:56Z) Update PR data Daily collection (2025-10-03T01:16:30Z) Oct 3, 2025
@gounthar gounthar force-pushed the auto-update-pr-data branch from a447a7e to 405de66 Compare October 3, 2025 01:16
@gounthar gounthar changed the title Update PR data Daily collection (2025-10-03T01:16:30Z) Update PR data Daily collection (2025-10-04T01:13:24Z) Oct 4, 2025
@gounthar gounthar force-pushed the auto-update-pr-data branch from 405de66 to a2352b9 Compare October 4, 2025 01:13

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 405de66 and a2352b9.

📒 Files selected for processing (2)
  • data/consolidated/all_prs.json (1 hunks)
  • data/consolidated/open_prs.json (1 hunks)
✅ Files skipped from review due to trivial changes (1)
  • data/consolidated/all_prs.json

Comment on lines +1406 to +1542
"completedAt": "2025-10-03T20:06:15Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://github.com/jenkinsci/appscan-plugin/runs/50532494517",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/cpd",
"name": "CPD",
"startedAt": "2025-10-03T20:06:20Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:13Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/checkstyle",
"name": "CheckStyle",
"startedAt": "2025-10-03T20:06:18Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:01Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3//coverage",
"name": "Code Coverage",
"startedAt": "2025-10-03T20:06:06Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:16Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://github.com/jenkinsci/appscan-plugin/runs/51919328998",
"name": "Jenkins Security Scan",
"startedAt": "2025-09-16T20:48:10Z",
"startedAt": "2025-10-03T20:06:14Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:05Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/eslint",
"name": "ESLint",
"startedAt": "2025-10-03T20:06:10Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:06Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/java",
"name": "Java Compiler",
"startedAt": "2025-10-03T20:06:11Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:07Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/javadoc-warnings",
"name": "JavaDoc",
"startedAt": "2025-10-03T20:06:12Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:11:50Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/display/redirect",
"name": "Jenkins",
"startedAt": "2025-10-03T20:04:59Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:04Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/maven-warnings",
"name": "Maven",
"startedAt": "2025-10-03T20:06:09Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:17Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/open-tasks",
"name": "Open Tasks Scanner",
"startedAt": "2025-10-03T20:06:22Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:14Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/pmd",
"name": "PMD",
"startedAt": "2025-10-03T20:06:19Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:06:11Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/spotbugs",
"name": "SpotBugs",
"startedAt": "2025-10-03T20:06:16Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:05:58Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/display/redirect?page=tests",
"name": "Tests / linux-21 / Build (linux-21)",
"startedAt": "2025-10-03T20:06:03Z",
"status": "COMPLETED",
"workflowName": ""
},
{
"__typename": "CheckRun",
"completedAt": "2025-10-03T20:11:47Z",
"conclusion": "SUCCESS",
"detailsUrl": "https://ci.jenkins.io/job/Plugins/job/appscan-plugin/job/PR-250/3/display/redirect?page=tests",
"name": "Tests / windows-17 / Build (windows-17)",
"startedAt": "2025-10-03T20:11:52Z",
"status": "COMPLETED",
"workflowName": ""

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Fix inverted CI timestamps.

Several check-run entries (e.g., “CPD”, “CheckStyle”, “Code Coverage”, “Jenkins”, and “Tests / windows-17 / Build (windows-17)”) have startedAt values that are later than their completedAt values. That temporal inversion makes the telemetry invalid and breaks downstream consumers that assume start ≤ completion. Please correct the timestamps before merging.

🤖 Prompt for AI Agents
In data/consolidated/open_prs.json around lines 1406 to 1542 several CheckRun
objects (examples: "CPD", "CheckStyle", "Code Coverage", "Jenkins", "Tests /
windows-17 / Build (windows-17)") have startedAt timestamps later than
completedAt; find each offending entry and correct the timestamps so startedAt ≤
completedAt by either swapping the two values when they were inverted or setting
startedAt to a sensible earlier time (e.g., completedAt minus a small delta)
while preserving ISO8601/Z timezone formatting; apply the same fix consistently
to all affected check-run entries and run the JSON timestamp validator to ensure
no further inversions remain before merging.

@gounthar gounthar merged commit 94a18ef into main Oct 4, 2025
7 of 8 checks passed
@gounthar gounthar deleted the auto-update-pr-data branch October 4, 2025 07:57
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.

1 participant