Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Add missing name field to Pipelines struct #1834

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

ppeble
Copy link
Contributor

@ppeble ppeble commented Nov 9, 2023

This is used in the following functions:

  • GetPipeline
  • GetLatestPipeline
  • CreatePipeline
  • RetryPipelineBuild
  • CancelPipelineBuild

GetPipeline and GetLatestPipeline return the name but the Create, Retry, and Cancel do not. But this will behave the same as DetailedStatus does and just not be set if it's not returned.

API docs for pipelines: https://docs.gitlab.com/ee/api/pipelines.html

Example JSON from GitLab for get latest pipeline:

{
    "id": 287,
    "iid": 144,
    "project_id": 21,
    "name": "Build pipeline",
    "sha": "50f0acb76a40e34a4ff304f7347dcc6587da8a14",
    "ref": "main",
    "status": "success",
    "source": "push",
    "created_at": "2022-09-21T01:05:07.200Z",
    "updated_at": "2022-09-21T01:05:50.185Z",
    "web_url": "http://127.0.0.1:3000/test-group/test-project/-/pipelines/287",
    "before_sha": "8a24fb3c5877a6d0b611ca41fc86edc174593e2b",
    "tag": false,
    "yaml_errors": null,
    "user": {
        "id": 1,
        "username": "root",
        "name": "Administrator",
        "state": "active",
        "avatar_url": "https://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=80&d=identicon",
        "web_url": "http://127.0.0.1:3000/root"
    },
    "started_at": "2022-09-21T01:05:14.197Z",
    "finished_at": "2022-09-21T01:05:50.175Z",
    "committed_at": null,
    "duration": 34,
    "queued_duration": 6,
    "coverage": null,
    "detailed_status": {
        "icon": "status_success",
        "text": "passed",
        "label": "passed",
        "group": "success",
        "tooltip": "passed",
        "has_details": false,
        "details_path": "/test-group/test-project/-/pipelines/287",
        "illustration": null,
        "favicon": "/assets/ci_favicons/favicon_status_success-8451333011eee8ce9f2ab25dc487fe24a8758c694827a582f17f42b0a90446a2.png"
    }
}

This also fixes a go fmt thing? I just followed the contributing guide and ran it and it produced this. 😄

Copy link
Member

@svanharmelen svanharmelen left a comment

Choose a reason for hiding this comment

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

Thanks @ppeble 👍🏻

@svanharmelen svanharmelen merged commit d906aaa into xanzy:main Nov 13, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants