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

Replace YAML.cpp with Rapid Yaml library #344

Closed
jj16791 opened this issue Nov 3, 2023 · 0 comments · Fixed by #343
Closed

Replace YAML.cpp with Rapid Yaml library #344

jj16791 opened this issue Nov 3, 2023 · 0 comments · Fixed by #343
Assignees
Labels
0.9.6 Part of SimEng Release 0.9.6 enhancement New feature or request

Comments

@jj16791
Copy link
Contributor

jj16791 commented Nov 3, 2023

Due to linking issues with the SST framework, replacing the current YAML.cpp dependancy with a header only yaml library (such as rapid yaml) will allow for easier integration with SST

@jj16791 jj16791 converted this from a draft issue Nov 3, 2023
@jj16791 jj16791 self-assigned this Nov 3, 2023
@jj16791 jj16791 added the enhancement New feature or request label Nov 3, 2023
@jj16791 jj16791 linked a pull request Nov 3, 2023 that will close this issue
@FinnWilkinson FinnWilkinson moved this from In Development to In Review in SimEng Development Nov 3, 2023
@FinnWilkinson FinnWilkinson moved this from In Review to Done in SimEng Development Nov 7, 2023
@FinnWilkinson FinnWilkinson moved this from Done to Changes Requested in SimEng Development Nov 7, 2023
@FinnWilkinson FinnWilkinson moved this from Changes Requested to Approved to Merge in SimEng Development Nov 27, 2023
@FinnWilkinson FinnWilkinson added the 0.9.6 Part of SimEng Release 0.9.6 label Dec 14, 2023
@jj16791 jj16791 moved this from Approved to Merge to Done in SimEng Development Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.9.6 Part of SimEng Release 0.9.6 enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants