vscode-ceedling-test-adapter icon indicating copy to clipboard operation
vscode-ceedling-test-adapter copied to clipboard

Feat additionnal project yaml

Open JuPrgn opened this issue 1 year ago • 3 comments

Fix #117

Add a feature to use Ceedling project: argument in order to allow to merge/load custom configuration from an additional project_specific.yml to the main project.yml.

This is usefull to test on different compiler or for different hardware as you can keep you common settings on main project.yml and use multiple specific.yml files to changes flags, defines... at runtime.

JuPrgn avatar Dec 21 '23 15:12 JuPrgn

The data in project.yml is also read by the extension and any additional .yml file merged at runtime should be considered the same.

  • The translation from the option names to .yml files should be done using the [:project][:options_paths] setting in project.yml
  • The option files should be read and merged in getYmlProjectData()
  • The option files should be watched for change by watchFilesForReload()

numaru avatar Jan 02 '24 20:01 numaru

Thank you for the contribution. I prefer to not include the PR in today's release because I think we can improve the support for options files and I don't want to change the ceedlingExplorer settings too often.

numaru avatar Jan 02 '24 20:01 numaru

Thank you for your feedback I will improve this PR soon following your recommendations. If I understand it well you propose to merge optional data to a single project.yml that will be used by getYmlProjectData() ? Merging project.yml and optional.yml datas could be an issue if some key are defined twice ? this is a check that is performed by the extension right ?

JuPrgn avatar Jan 02 '24 20:01 JuPrgn