search

swiftlang / swift-driver

Swift compiler driver reimplementation in Swift
Apache License 2.0
795 stars 192 forks source link

Remove a dependency on Yams by encoding the file list in JSON #1701

Open jakepetroules opened 1 week ago

jakepetroules commented 1 week ago

YAML is a strict superset of JSON, so this should not have compatibility implications.

jakepetroules commented 1 week ago

@swift-ci please test

jakepetroules commented 1 week ago

@artemcm Is this change OK? Does running the test suite provide sufficient coverage or is there anything else we can do to verify the other end is still able to consume the data?

artemcm commented 1 week ago

@artemcm Is this change OK? Does running the test suite provide sufficient coverage or is there anything else we can do to verify the other end is still able to consume the data?

The driver's own test suite should be okay. I'd also launch PR testing with a mention of this PR from the main swift repo.