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

Ensure all standard configs are parsed with the ConfigParser class #94

Open
TB-1993 opened this issue Sep 16, 2024 · 0 comments
Open

Ensure all standard configs are parsed with the ConfigParser class #94

TB-1993 opened this issue Sep 16, 2024 · 0 comments
Milestone

Comments

@TB-1993
Copy link
Contributor

TB-1993 commented Sep 16, 2024

Description:

Problem/Opportunity: Currently the device_config.yml file is parsed by the core ConfigParser class, however the rack_config.yml is parsed by the pyYaml libraries load function. This could cause differences in behaviour when parsing config files and would make it harder to change to different config file format in the future.

Proposed Solution: The entire core framework should be reviewed to ensure all config files are parsed by the ConfigParser class. Configs parsed using the pyYaml load method will need to be checked to ensure they can still be used as they were before.

Acceptance Criteria: (Optional)

  • All config files loaded as part of the core framework are loaded by the ConfigParser class
  • No breaking behaviour is introduced after changing the parser.
@Ulrond Ulrond added this to the future milestone Oct 22, 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

No branches or pull requests

2 participants