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

Config refactor #34

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Commits on Feb 25, 2022

  1. refactored cmd

    * moved `~/cmd/app.go` to `~/cmd/my5g-RANTester/main.go`
    * refactored logging, command declaration
    * split into multiple files
    * switched to using constants and fmt.Sprintf for formatting stuff
    gravestench committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    1e13109 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    78c3216 View commit details
    Browse the repository at this point in the history

Commits on Mar 2, 2022

  1. refactored config, added cli options

    * config global state removed from the config module
    * config global state now resides in main, TODO to remove it
    * config now creates a default if none are found
    * added command line option for specifying the config file
    * added command line option to specify the log level
    * config directory now looks in user dir, in a platform agnostic way.
    ex: on linux, will typically look in `~/.config/my5g/RANTester` for file
    `config.yml`
    * updated go.mod to use go v1.16
    * embedded default config file data
    * errors are now bubbled up appropriately
    * depends on my5G#33
    gravestench committed Mar 2, 2022
    Configuration menu
    Copy the full SHA
    0108ab1 View commit details
    Browse the repository at this point in the history