Skip to content

Exit code now reflects exactly the type of error - #6

Merged
brandonc merged 3 commits into
mainfrom
TF-35990-implement-detailed-exit-codes-across-the-cli
Apr 14, 2026
Merged

Exit code now reflects exactly the type of error#6
brandonc merged 3 commits into
mainfrom
TF-35990-implement-detailed-exit-codes-across-the-cli

Conversation

@brandonc

Copy link
Copy Markdown
Collaborator

0 = OK
1 = Usage error
2 = Not Found error (or not authorized)
3 = Authentication error
4 = Network error
5 = API server error (after retries failed)

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

If you have any questions, please contact your direct supervisor, GRC (#team-grc), or the PCI working group (#proj-pci-reboot). You can also find more information at PCI Compliance.

Screenshot 2026-04-14 at 16 39 56

0 = OK
1 = Usage error
2 = Not Found error (or not authorized)
3 = Authentication error
4 = Network error
5 = API server error (after retries failed)
@github-actions

Copy link
Copy Markdown
Contributor

Reliability Check(s) Results

Target Asset: tfcloud
Checks Executed: none

✅ All checks passed!

Comment thread README.md Outdated

| Exit | Meaning | Solution |
|------|----------------------------------|-----------------------------|
| 0 | OK | &emdash; |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this is not showing up in MD btw

@brandonc
brandonc merged commit 5167aaf into main Apr 14, 2026
3 checks passed
@brandonc
brandonc deleted the TF-35990-implement-detailed-exit-codes-across-the-cli branch April 14, 2026 23:06
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.

2 participants