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

FSOC-37 Added argument support to fsoc config set #132

Merged
merged 2 commits into from
Jul 9, 2023

Conversation

GDW1
Copy link
Contributor

@GDW1 GDW1 commented Jun 27, 2023

Description

Addressing FSOC-37

Before you were only able to provide values for the config file through flags (i.e. "--auth oauth"). Now you can provide values for the config file through arguments (i.e. "auth=oauth")

This is closer to git config style

This is done with little refactoring to the existent code and all changes still support the old way of supplying values (which can be mixed and matched i.e. "auth=oauth --url URL")

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@GDW1 GDW1 requested a review from pnickolov as a code owner June 27, 2023 18:36
@codecov-commenter
Copy link

codecov-commenter commented Jun 27, 2023

Codecov Report

Merging #132 (7d9be89) into main (62d908f) will decrease coverage by 0.26%.
The diff coverage is 0.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
- Coverage   32.34%   32.09%   -0.26%     
==========================================
  Files          27       27              
  Lines        3568     3596      +28     
==========================================
  Hits         1154     1154              
- Misses       2323     2351      +28     
  Partials       91       91              
Impacted Files Coverage Δ
cmd/config/set.go 8.06% <0.00%> (-1.43%) ⬇️

@pnickolov pnickolov merged commit 113e56b into cisco-open:main Jul 9, 2023
7 checks passed
GDW1 pushed a commit to GDW1/fsoc that referenced this pull request Jul 10, 2023
* FSOC-37 Added argument support to

* fixed deprecation messages, fixed documentation examples

---------

Co-authored-by: GDW1 <guwilks@cisco.com>
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