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

Add a new configuration package #1868

Closed
wants to merge 6 commits into from
Closed

Add a new configuration package #1868

wants to merge 6 commits into from

Conversation

kislaykishore
Copy link
Collaborator

@kislaykishore kislaykishore commented Apr 26, 2024

  • This package brings in support for both flags as well as config files to configure GCSFuse.
  • It'll be integrated into GCSFuse in subsequent commits and will replace the dated urfave/cli library with spf13/viper.

Description

Link to the issue in case of a bug fix.

NA

Testing details

  1. Manual - NA
  2. Unit tests - Added new unit tests for each param
  3. Integration tests - NA

* This module brings in support for both flags as well as config files
  to configure GCSFuse.
* It'll be integrated into GCSFuse in subsequent commits and will
  replace the dated urfave/cli library with spf13/viper.
@kislaykishore kislaykishore requested a review from a team as a code owner April 26, 2024 11:19
@kislaykishore kislaykishore changed the title Add a new configuration module Add a new configuration package Apr 26, 2024
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.

None yet

1 participant