Skip to content
This repository has been archived by the owner on Nov 22, 2022. It is now read-only.

fix(ci trace): exit normally when prompts are interrupted #849

Merged
merged 1 commit into from
Oct 3, 2021

Conversation

hoelzro
Copy link
Contributor

@hoelzro hoelzro commented Oct 1, 2021

鈥pecifically, interrupted via SIGINT (CTRL-C). Otherwise,
the first job is chosen, which isn't a huge deal, but isn't really
consistent with the user's wish to exit the program early

Description

I noticed that if I do glab ci trace and CTRL-C to exit, the first job gets chosen, rather than exiting.

Related Issue

(none to my knowledge)

How Has This Been Tested?

I've only done manual testing so far; I'd run the automatic test suite but I don't want to blow my configuration away 馃槵

Screenshots (if appropriate):

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)

鈥pecifically, interrupted via SIGINT (CTRL-C).  Otherwise,
the first job is chosen, which isn't a huge deal, but isn't really
consistent with the user's wish to exit the program early
@codecov
Copy link

codecov bot commented Oct 2, 2021

Codecov Report

Merging #849 (fe0056a) into trunk (5cb2858) will decrease coverage by 0.03%.
The diff coverage is 20.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            trunk     #849      +/-   ##
==========================================
- Coverage   59.42%   59.38%   -0.04%     
==========================================
  Files          93       93              
  Lines        7366     7370       +4     
==========================================
  Hits         4377     4377              
- Misses       2607     2610       +3     
- Partials      382      383       +1     
Impacted Files Coverage 螖
commands/ci/trace/trace.go 56.97% <20.00%> (-2.78%) 猬囷笍

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update fe01cfc...fe0056a. Read the comment docs.

Copy link
Owner

@profclems profclems left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for working on this @hoelzro

@profclems profclems merged commit 2779151 into profclems:trunk Oct 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants