Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add IOStreams inside command/io and add command/args package #249

Merged
merged 1 commit into from
Aug 22, 2022

Conversation

danielfbm
Copy link
Contributor

Changes

With IOStreams from k8s.io/cli-runtime creates a better methods of
providing data streams to cli while making it easy to create tests
inside those packages

command/args is a set of methods to read and extract data from cli
arguments making it easy to integrate into other clis and into task
scripts

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • spec PR link included
  • Follows the commit message standard
  • Meets the contributing guidelines (including
    functionality, content, code)
  • Test cases with documentation and functionality works as expected using current and related github repos (MUST deploy and check)
  • Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes

@danielfbm danielfbm force-pushed the feat/add-iostreams-command-args branch from 82ca083 to b04e6e2 Compare August 22, 2022 01:26
@alaudabot
Copy link
Contributor

alaudabot commented Aug 22, 2022

Quality Gate

updated at Mon Aug 22 02:25:42 UTC 2022

Status: OK 🟢

Metric Rating
🐛 Bugs: 0 🏗 Reliability: A 🟢
🤮 Code smells: 0 🛠 Maintainability: A 🟢
⚠️ Vulnerabilities: 0 🦺 Security: A 🟢
🔥 Security hotspots: 0 🔎 Security Review: A 🟢
Metric Change set Target (main) After
🧪 Code coverage 100.00%
Coverage on 46 new lines
41.30% 41.70% ⬆️ 🟢
estimated after merge
allowed difference 0
👥 Duplications 0.00%
Duplications on 0 new lines
0.20% 0.20% ↔️ 🟢
estimated after merge
allowed difference 0

[249]

@danielfbm danielfbm force-pushed the feat/add-iostreams-command-args branch 2 times, most recently from dd555ed to 690702f Compare August 22, 2022 01:36
nanjingfm
nanjingfm previously approved these changes Aug 22, 2022
With IOStreams from k8s.io/cli-runtime creates a better methods of
providing data streams to cli while making it easy to create tests
inside those packages

command/args is a set of methods to read and extract data from cli
arguments making it easy to integrate into other clis and into task
scripts
@danielfbm danielfbm merged commit b55afc8 into main Aug 22, 2022
@danielfbm danielfbm deleted the feat/add-iostreams-command-args branch August 22, 2022 02:27
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