Get the title of the revision instead of the commit
ClosedPublic
Actions

bin/publish-latest-branch-build.php
248	Nice! Small thing: line 250 may be null, should be something like: $revision_title = $result['data'][0]['fields']['revision_title'] ?? $git_commit_title;

valerio.bozzolan added a reviewer: Libre BusTO CI/CD Blessed Reviewers.Tue, Apr 28, 12:29

Restricted Application added a blocking reviewer: Libre BusTO CI/CD Blessed Reviewers. · View Herald TranscriptTue, Apr 28, 12:29

fix AI allucionation about field name, fix possible null pointer

Harbormaster completed remote builds in B847: Diff 717.Tue, Apr 28, 12:36

valerio.bozzolan added inline comments.Tue, Apr 28, 12:37

bin/publish-latest-branch-build.php

248

Quasi giusto, cambio il campo da 'revision_title' a 'title' e dovrebbe andare

Esempio output

{
  "data": [
    {
      "id": 233,
      "type": "DREV",
      "phid": "PHID-DREV-qe4xbgymd757kx4dau5c",
      "fields": {
        "title": "Get the title of the revision instead of the commit",
        "uri": "https://gitpull.it/D233",
        "authorPHID": "PHID-USER-kur5pqwjhjcmjnjluwil",
        "status": {
          "value": "needs-review",
          "name": "Needs Review",
          "closed": false,
          "color.ansi": "magenta"
        },
        "repositoryPHID": "PHID-REPO-2erlvbhmi3uloi4ubzph",
        "diffPHID": "PHID-DIFF-ipybxgjc5fsru44fxlqj",
        "summary": "",
        "testPlan": "",
        "isDraft": false,
        "holdAsDraft": false,
        "dateCreated": 1777371557,
        "dateModified": 1777372193,
        "policy": {
          "view": "public",
          "edit": "users"
        }
      },
      "attachments": {}
    }
  ],
  "maps": {},
  "query": {
    "queryKey": null
  },
  "cursor": {
    "limit": 100,
    "after": null,
    "before": null,
    "order": null
  }
}

valerio.bozzolan accepted this revision.Tue, Apr 28, 12:37

This revision is now accepted and ready to land.Tue, Apr 28, 12:37

Closed by commit R68:304f365d8644: Get the title of the revision instead of the commit. · Explain WhyTue, Apr 28, 12:37

This revision was automatically updated to reflect the committed changes.

valerio.bozzolan added a commit: R68:304f365d8644: Get the title of the revision instead of the commit.

valerio.bozzolan added a task: T1281: Telegram bot: show Differential diff name (not last commit title).Tue, Apr 28, 12:46