Skip to content

When viewing a pipeline, print the corresponding job ids#108

Merged
andrew merged 1 commit into
git-pkgs:mainfrom
vvsagar:feature/show-job-ids-in-ci-view
Jun 2, 2026
Merged

When viewing a pipeline, print the corresponding job ids#108
andrew merged 1 commit into
git-pkgs:mainfrom
vvsagar:feature/show-job-ids-in-ci-view

Conversation

@vvsagar

@vvsagar vvsagar commented May 27, 2026

Copy link
Copy Markdown
Contributor

I was testing on a gitlab project and I wanted to see the log of a CI.

forge ci list first shows me a list of latest pipelines with their ids. forge ci log does not work directly on them, instead I need to forge ci view on the pipeline id, which makes sense.

But now I made forge ci view print the corresponding job ids, which I can then pass on to forge ci log and it works.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the human-readable forge ci view <run-id> output so users can see job IDs and pass them directly to forge ci log <job-id>.

Changes:

  • Adds each CI job’s numeric ID to the Jobs section printed by ci view.
  • Keeps existing status selection behavior using Conclusion when available, otherwise Status.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@andrew andrew 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.

Makes sense — ci view → grab job ID → ci log is the obvious flow and this unblocks it.

Column alignment will drift when job IDs differ in width, but the existing output already has that with job names so not a regression.

@andrew andrew merged commit bc40864 into git-pkgs:main Jun 2, 2026
4 checks passed
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.

3 participants