Skip to content

feat: add environment column to --list and --list-csv (#1837)#1838

Open
bcouetil wants to merge 1 commit intofirecow:masterfrom
bcouetil:feat/environment-in-list
Open

feat: add environment column to --list and --list-csv (#1837)#1838
bcouetil wants to merge 1 commit intofirecow:masterfrom
bcouetil:feat/environment-in-list

Conversation

@bcouetil
Copy link
Copy Markdown
Collaborator

@bcouetil bcouetil commented Apr 20, 2026

Display the environment name in --list and --list-csv outputs, placed just before the needs column. Jobs without an environment show an empty value. The list-environment test expects workflow:rules:variables resolution (PR #1833) and will fail until that feature is merged.

Closes #1837

Screenshot 2026-04-20 at 16 06 57

Summary by cubic

Add an environment column to --list and --list-csv, placed before the needs column, so you can see each job’s target environment. Jobs without an environment show a blank value.

  • New Features
    • --list: adds an "environment" column and prints job.environment.name (blank if none).
    • --list-csv: adds an "environment" field after allowFailure.
    • Tests: updates existing list outputs and adds a list-environment case. The list-environment test depends on workflow:rules:variables resolution (see PR feat: implement workflow:rules:variables support (#1832) #1833) to resolve $DEPLOY_ENV.

Written for commit b0a6cdb. Summary will update on new commits.

@bcouetil bcouetil requested a review from firecow April 20, 2026 14:07
@bcouetil bcouetil self-assigned this Apr 20, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

@bcouetil bcouetil force-pushed the feat/environment-in-list branch from b0a6cdb to 926cb3e Compare April 30, 2026 22:04
Display the environment name in --list and --list-csv outputs, placed
just before the needs column. Jobs without an environment show an empty
value. The list-environment test expects workflow:rules:variables
resolution (PR firecow#1833) and will fail until that feature is merged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bcouetil bcouetil force-pushed the feat/environment-in-list branch from 926cb3e to e1fd4f0 Compare April 30, 2026 22:18
@bcouetil
Copy link
Copy Markdown
Collaborator Author

Since you merged the #1833 @firecow , I've rebase and fixed tests.

I think my work here is done too 😊

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.

Add environment name to --list and --list-csv

1 participant