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

chore(cli): introduce EOL message #9741

Merged
merged 2 commits into from Sep 7, 2023
Merged

chore(cli): introduce EOL message #9741

merged 2 commits into from Sep 7, 2023

Conversation

odubajDT
Copy link
Contributor

@odubajDT odubajDT commented Sep 7, 2023

This PR

  • adds EOL message to each CLI command

Related Issues

Part of: #9733

How to test

  1. Build cli with go build . -o keptn
  2. Execute any command, for example ./keptn auth

Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT odubajDT requested a review from a team as a code owner September 7, 2023 13:08
@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Merging #9741 (29046d2) into master (e332b8a) will decrease coverage by 0.03%.
Report is 1 commits behind head on master.
The diff coverage is 44.44%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9741      +/-   ##
==========================================
- Coverage   70.91%   70.88%   -0.03%     
==========================================
  Files         618      618              
  Lines       32731    32738       +7     
  Branches     1645     1645              
==========================================
- Hits        23210    23207       -3     
- Misses       8276     8283       +7     
- Partials     1245     1248       +3     
Files Changed Coverage Δ
cli/cmd/root.go 71.13% <0.00%> (-0.75%) ⬇️
cli/pkg/config/cli_config.go 55.68% <50.00%> (-0.42%) ⬇️

... and 1 file with indirect coverage changes

Flag Coverage Δ
cli 44.46% <44.44%> (-0.14%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@sonarcloud
Copy link

sonarcloud bot commented Sep 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@odubajDT odubajDT merged commit 4eb928e into master Sep 7, 2023
21 of 22 checks passed
@odubajDT odubajDT deleted the chore/9733/eol-msg branch September 7, 2023 14:04
@github-actions github-actions bot mentioned this pull request Sep 7, 2023
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.

None yet

2 participants