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

Parse scenariofile in constructor and adding helper function to deserialize std::variant-like types. #100

Merged
merged 2 commits into from
Sep 1, 2023

Conversation

ltoenning
Copy link
Member

Proposed Changes

For simplification and to avoid the use of wrong data from the scenariofile parser: Directly parse the scenariofile in the constructor.

This PR also adds a function to deserialize YAML entries into std::variant-like types. This will be used in the future for a type-safe list of all different scenario sequence steps.

Definition of Done (optional)

  • Feature is implemented
  • No known bugs that stops the correct execution
  • CI / CD pipeline passed
  • The code guidelines were followed
  • [ ] Unit and / or Integration tests for the new feature
  • [ ] New feature was added to the documentation

@ltoenning ltoenning self-assigned this Aug 31, 2023
@ltoenning ltoenning marked this pull request as draft August 31, 2023 11:38
@ltoenning ltoenning marked this pull request as ready for review August 31, 2023 13:22
@ltoenning ltoenning merged commit 85f6072 into iml130:main Sep 1, 2023
5 checks passed
@ltoenning ltoenning deleted the scenariofileparser branch September 1, 2023 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants